Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

README should say "Authentication", not "Authorization" #841

Closed
dhermes opened this issue Sep 1, 2015 · 3 comments
Closed

README should say "Authentication", not "Authorization" #841

dhermes opened this issue Sep 1, 2015 · 3 comments
Assignees

Comments

@dhermes
Copy link
Contributor

dhermes commented Sep 1, 2015

I'm kind of on the fence if I even believe this to be true or not.

The OAuth 2.0 flows themselves are authorization (i.e. authorizing the clients to act on behalf of the user) but the service account flows are user-independent and really serve to authenticate an entity with a given access.

Also, when we use non-service accounts we just use the Application Default Credentials from the gcloud CLI. These will already have a refresh token, so again we perform / participate in no authorization, just authentication.


PS Sorry for the esoteric issue, but I was adding this section to the gcloud-python README and it just occurred to me.


PPS Is there some magic trick to get a link to lines in a MD file on GitHub?

@callmehiphop
Copy link
Contributor

Your logic seems sound to me!

I wish there was a trick for that, the closest thing I know of is linking to sections, e.g. https://github.com/GoogleCloudPlatform/gcloud-node/blob/master/README.md#authorization

@dhermes
Copy link
Contributor Author

dhermes commented Sep 2, 2015

I went to create a PR this morning when filing this issue when I realized I don't have a fork. @callmehiphop shall I create one?

@callmehiphop
Copy link
Contributor

@dhermes If you don't mind, that would be awesome!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants