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

New resource: sendgrid_api_key #3

Merged
merged 3 commits into from
Feb 15, 2021

Conversation

davidji99
Copy link
Owner

@davidji99 davidji99 commented Feb 14, 2021

Resolves #2

@davidji99
Copy link
Owner Author

davidji99 commented Feb 14, 2021

If you create an API with no scope using an API key that has XYZ scopes, that new key wlll only have XYZ scopes.

If you create an API with scopes AB using an API key that only has XYZ scopes, the operation will fail. The new key can only have all or a subset of whatever scopes the authenticated key has.

For updates:

  • PUT requires scopes to be set. This is problematic if you wanted to just change the name. It also requires the name to be set even if you're just changing scopes.
  • PUT overwrites scopes with whatever you have set in configuration.

@davidji99 davidji99 force-pushed the issues/2/davidji99-initial-bootstrapping branch from dd2a9f2 to df6a6ce Compare February 15, 2021 00:41
@davidji99 davidji99 changed the title Issues/2/davidji99 initial bootstrapping New resource: sendgrid_api_key Feb 15, 2021
@davidji99 davidji99 merged commit 5a02666 into master Feb 15, 2021
@davidji99 davidji99 deleted the issues/2/davidji99-initial-bootstrapping branch February 15, 2021 00:52
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.

New data source: sendgrid_api_key
1 participant