Commite96764d

committed

Update scopes check code sample

The customer in#820 was trying to clarify when they should do the "scopes check" we suggest. I updated the code sample to be within the callback which I hope clarifies the flow a bit.cc@github/platform @github/docs-platform

1 parent268d647 commite96764dCopy full SHA for e96764d

File tree

1 file changed

-3

lines changed

content/guides
- basics-of-authentication.md

1 file changed

-3

lines changed

`‎content/guides/basics-of-authentication.md‎`

Lines changed: 9 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -125,9 +125,15 @@ The scopes that were granted are returned as a part of the response from`
`125`	`125`	`exchanging a token.`
`126`	`126`
`127`	`127`	`#!ruby`
`128`		`-# check if we were granted user:email scope`
`129`		`-scopes = JSON.parse(result)['scope'].split(',')`
`130`		`-has_user_email_scope = scopes.include? 'user:email'`
	`128`	`+get '/callback' do`
	`129`	`+ # ...`
	`130`	`+ # Get the access_token using the code sample above`
	`131`	`+ # ...`
	`132`	`+`
	`133`	`+ # check if we were granted user:email scope`
	`134`	`+ scopes = JSON.parse(result)['scope'].split(',')`
	`135`	`+ has_user_email_scope = scopes.include? 'user:email'`
	`136`	`+end`
`131`	`137`
`132`	`138`	In our application, we're using`scopes.include?` to check if we were granted
`133`	`139`	the`user:email` scope needed for fetching the authenticated user's private

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commite96764d

File tree

1 file changed

1 file changed

`‎content/guides/basics-of-authentication.md‎`

0 commit comments