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

fix: Fix entrypoint housekeeping api key scope format #449

Merged
merged 1 commit into from
Apr 9, 2024

Conversation

lotooo
Copy link
Contributor

@lotooo lotooo commented Feb 16, 2024

Description

I recently upgraded our setup to docker-web:9.0.1 and discovered that my previous commit on the entrypoint broke it and was generating a housekeeping api key with wrong scope (every scope was surrounded by double quotes).
This PR fixes my mistake

Changes

Remove the quotes from the scopes that were introduced in a584da5.

Checklist

  • Pull request is limited to a single purpose
  • Code style/formatting is consistent
  • All existing tests are passing
  • Added new tests related to change
  • No unnecessary whitespace changes

Collaboration
When a user creates a pull request from a fork that they own, the user
generally has the authority to decide if other users can commit to the
pull request's compare branch. If the pull request author wants greater
collaboration, they can grant maintainers of the upstream repository
(that is, anyone with push access to the upstream repository) permission
to commit to the pull request's compare branch

See https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/allowing-changes-to-a-pull-request-branch-created-from-a-fork

@satterly satterly merged commit a12f3e0 into alerta:master Apr 9, 2024
2 checks passed
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