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.

Sign up for GitHub

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

Jump to bottom

fix documentation related to authentication strategy #1085

Closed
sofisl opened this issue Oct 21, 2020 · 1 comment · Fixed by #1653
Closed

fix documentation related to authentication strategy #1085

sofisl opened this issue Oct 21, 2020 · 1 comment · Fixed by #1653
Assignees
@bcoe
Labels
type: docs Improvement to the documentation for an API. type: process A process-related concern. May include testing, release, or the like.

Comments

@sofisl
Copy link
Contributor

sofisl commented Oct 21, 2020

Related to #1084 and #983

How, and in what order of operations, are we authenticating in this library? We have a document describing how environment variables, constructors, etc., operate together, but what about order of operations within the constructor? For example, where do keyfile, credentials, andclientOptions fit in against the wider order of operations for authentication? When should we be trying to get a projectId?
@sofisl sofisl added type: process A process-related concern. May include testing, release, or the like. type: docs Improvement to the documentation for an API. labels Oct 21, 2020
gcf-owl-bot bot added a commit that referenced this issue Jan 13, 2022
@gcf-owl-bot
chore: update github issue templates (#1085)
70c05f9 
Source-Link: googleapis/synthtool@4dfd20a
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-nodejs:latest@sha256:2d850512335d7adca3a4b08e02f8e63192978aea88c042dacb3e382aa996ae7c
gcf-merge-on-green bot pushed a commit that referenced this issue Jan 13, 2022
@gcf-owl-bot
chore: update github issue templates (#1085) (#1347)
097d328
@champ-devops champ-devops mentioned this issue Sep 6, 2022
Closed
@danielbankhead danielbankhead mentioned this issue Sep 29, 2023
Merged
@danielbankhead
Copy link
Contributor

The projectId part of this request has been partially resolved here (essentially, we'll do a best effort as some APIs do not require it):

For the others, I raised a PR to improve the documentation: #1653

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bcoe bcoe

Labels
type: docs Improvement to the documentation for an API. type: process A process-related concern. May include testing, release, or the like.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

docs(GoogleAuth): Document Credential Precedence googleapis/google-auth-library-nodejs
3 participants
@bcoe @danielbankhead @sofisl