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

Fixed the OAuth2TokenManager, which was not properly checking against unconfigured token request parameters #425

Merged
merged 1 commit into from
Oct 14, 2024

Conversation

cdavernas
Copy link
Member

@cdavernas cdavernas commented Oct 14, 2024

Many thanks for submitting your Pull Request ❤️!

What this PR does / why we need it:

Fixes the OAuth2TokenManager, which was not properly checking against unconfigured token request parameters

…erly checking against unconfigured token request parameters

Signed-off-by: Charles d'Avernas <charles.davernas@neuroglia.io>
@cdavernas cdavernas added type: bug Something isn't working app: runner Concerns the Runner labels Oct 14, 2024
@cdavernas cdavernas added this to the v1.0.0 milestone Oct 14, 2024
@cdavernas cdavernas self-assigned this Oct 14, 2024
@cdavernas cdavernas changed the title Fixed the OAuth2TokenManager, which was not prop…erly checking against unconfigured token request parameters Fixed the OAuth2TokenManager, which was not properly checking against unconfigured token request parameters Oct 14, 2024
@cdavernas cdavernas merged commit a611bcb into main Oct 14, 2024
4 checks passed
@cdavernas cdavernas deleted the fix-oauth2-request branch October 14, 2024 12:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
app: runner Concerns the Runner type: bug Something isn't working
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant