Skip to content

Scopes env var#4

Merged
egoodhall merged 3 commits intomasterfrom
scopes-env-var
Mar 12, 2019
Merged

Scopes env var#4
egoodhall merged 3 commits intomasterfrom
scopes-env-var

Conversation

@egoodhall
Copy link
Copy Markdown
Contributor

As was suggested in #1, this allows the scopes being requested by the quickstart app to be defined in an environment variable called SCOPES. Scopes can be separated by a comma, space, or url-encoded space (%20). Examples:

SCOPES='contacts,hubdb'
SCOPES='contacts hubdb'
SCOPES='contacts%20hubdb'

Also updates the docs to describe the scopes environment variable

Eric Marshall added 2 commits March 11, 2019 13:23
This allows the scopes being requested by the quickstart app to be
defined in an environment variable called SCOPES. Scopes can be comma,
space, or url-encoded space (%20) separated. Examples:

SCOPES='contacts,hubdb'
SCOPES='contacts hubdb'
SCOPES='contacts%20hubdb'
@chrisbaldauf
Copy link
Copy Markdown

nit: the URL param to kick off the authorization flow is scope, but the env var is scopes. Is there an opportunity for confusion there?

Otherwise LGTM

@egoodhall
Copy link
Copy Markdown
Contributor Author

Good catch - all the other env vars match up to query parameters, so I think that makes sense

@egoodhall egoodhall merged commit 00e306d into master Mar 12, 2019
@egoodhall egoodhall deleted the scopes-env-var branch March 12, 2019 12:57
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

Successfully merging this pull request may close these issues.

2 participants