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

Introduce circuit breaker for tokens with authorization problems #17387

Conversation

eduardoj
Copy link
Member

@eduardoj eduardoj commented Feb 14, 2025

Depends on #17380.

Mark the token as not enabled if we receive an unauthorized or a forbidden response.

@github-actions github-actions bot added the Frontend Things related to the OBS RoR app label Feb 14, 2025
@eduardoj eduardoj force-pushed the feature/workflow_token_circuit_breaker_automatic_revocation branch 3 times, most recently from 92ee001 to 001a43f Compare February 17, 2025 09:55
@eduardoj eduardoj marked this pull request as ready for review February 17, 2025 09:55
Copy link
Member

@hennevogel hennevogel left a comment

Choose a reason for hiding this comment

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

More stuff on top of SCMExceptionHandler but we are going to kill that 🐱 another day...

Mark the token as not enabled if we receive an unauthorized or a
forbidden response.
@eduardoj eduardoj force-pushed the feature/workflow_token_circuit_breaker_automatic_revocation branch from 001a43f to 0a1b8f6 Compare February 17, 2025 15:13
@hennevogel hennevogel merged commit 20c5ac9 into openSUSE:master Feb 18, 2025
21 checks passed
@eduardoj eduardoj deleted the feature/workflow_token_circuit_breaker_automatic_revocation branch February 18, 2025 10:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Frontend Things related to the OBS RoR app
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants