Skip to content

Conversation

@parthea
Copy link
Contributor

@parthea parthea commented Nov 12, 2020

Fixes #1071 🦕

Note: I changed scopes=['profile', 'email'], to scopes=['openid', 'https://www.googleapis.com/auth/userinfo.email', 'https://www.googleapis.com/auth/userinfo.profile'], in the examples because I received a warning that the scope has changed.

Warning: Scope has changed from "email profile" to "https://www.googleapis.com/auth/userinfo.email openid https://www.googleapis.com/auth/userinfo.profile"..

@parthea parthea requested a review from busunkim96 November 12, 2020 11:58
@parthea parthea requested a review from a team as a code owner November 12, 2020 11:58
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Nov 12, 2020
@parthea parthea changed the title Update oauth docs to include snippet to get email address of authenticated user docs: Update oauth docs to include snippet to get email address of authenticated user Nov 12, 2020
@parthea parthea force-pushed the update-oauth-doc-to-include-user-info branch from ca919fc to 0a27925 Compare November 12, 2020 12:20
Copy link

@tswast tswast left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! I didn't know about this sample, but I have to do the same by trial-and-error when I want a set of scopes to debug auth logic.

I'm a bit disappointed that this is a code sample embedded in a markdown file (not testable), but that's a whole other issue.

@parthea parthea added the automerge Merge the pull request once unit tests and other checks pass. label Nov 13, 2020
@gcf-merge-on-green gcf-merge-on-green bot merged commit 25fba64 into googleapis:master Nov 13, 2020
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Nov 13, 2020
@parthea parthea deleted the update-oauth-doc-to-include-user-info branch November 13, 2020 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add user_info api in the documentation

3 participants