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

backend-common: prevent expired backend tokens from being used #11807

Merged
merged 1 commit into from
Jun 3, 2022
Merged

backend-common: prevent expired backend tokens from being used #11807

merged 1 commit into from
Jun 3, 2022

Conversation

Rugvip
Copy link
Member

@Rugvip Rugvip commented Jun 2, 2022

Hey, I just made a Pull Request!

Followup for #11262, removing the deprecation and failing with an error if tokens are expired.

✔️ Checklist

  • A changeset describing the change and affected packages. (more info)
  • Added or updated documentation
  • Tests for new functionality and regression tests for bug fixes
  • Screenshots attached (for UI changes)
  • All your commits have a Signed-off-by line in the message. (more info)

@Rugvip
backend-common: prevent expired backend tokens from being used
55647ec 
Signed-off-by: Patrik Oldsberg <poldsberg@gmail.com>
@Rugvip Rugvip requested a review from a team as a code owner June 2, 2022 11:50
@github-actions
Copy link
Contributor

github-actions bot commented Jun 2, 2022

Changed Packages

Package Name Package Path Changeset Bump Current Version
@backstage/backend-common packages/backend-common minor v0.13.6-next.1

benjdlambert
benjdlambert approved these changes Jun 2, 2022
View reviewed changes
Copy link
Member

@benjdlambert benjdlambert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚢

@Rugvip Rugvip merged commit 8db77da into master Jun 3, 2022
@Rugvip Rugvip deleted the rugvip/exp branch June 3, 2022 14:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@benjdlambert benjdlambert benjdlambert approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Rugvip @benjdlambert