Skip to content

Do not validate auth mechanism if not present#15148

Merged
michaelklishin merged 1 commit intomainfrom
remove-unncessary-log-statement
Dec 16, 2025
Merged

Do not validate auth mechanism if not present#15148
michaelklishin merged 1 commit intomainfrom
remove-unncessary-log-statement

Conversation

@MarcialRosales
Copy link
Copy Markdown
Contributor

@MarcialRosales MarcialRosales commented Dec 16, 2025

Minor changes to oauth2 functionality in the management plugin.

Do not validate preferred_auth_mechanism or strict_auth_mechanism parameters if not present.
And do not log it when it is not present.

Types of Changes

What types of changes does your code introduce to this project?
Put an x in the boxes that apply

  • Bug fix (non-breaking change which fixes issue #NNNN)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause an observable behavior change in existing systems)
  • Documentation improvements (corrections, new content, etc)
  • Cosmetic change (whitespace, formatting, etc)
  • Build system and/or CI

@MarcialRosales MarcialRosales self-assigned this Dec 16, 2025
@MarcialRosales MarcialRosales changed the title Do not valid auth mechanism if not present Do not validate auth mechanism if not present Dec 16, 2025
@MarcialRosales MarcialRosales force-pushed the remove-unncessary-log-statement branch from d62b649 to 00fbea5 Compare December 16, 2025 13:08
@MarcialRosales MarcialRosales marked this pull request as ready for review December 16, 2025 13:54
@michaelklishin michaelklishin added this to the 4.3.0 milestone Dec 16, 2025
@michaelklishin michaelklishin merged commit b249dd0 into main Dec 16, 2025
290 checks passed
@michaelklishin michaelklishin deleted the remove-unncessary-log-statement branch December 16, 2025 15:58
michaelklishin added a commit that referenced this pull request Dec 17, 2025
4.2.3: Do not validate auth mechanism if not present (backport #15148)
michaelklishin added a commit that referenced this pull request Mar 30, 2026
OAuth 2 management UI: adopt a better approach than #15148
mergify bot pushed a commit that referenced this pull request Mar 30, 2026
michaelklishin added a commit that referenced this pull request Mar 31, 2026
OAuth 2 management UI: adopt a better approach than #15148 (backport #15887)
mergify bot pushed a commit that referenced this pull request Mar 31, 2026
(cherry picked from commit 5016060)
(cherry picked from commit f9e03d6)
michaelklishin added a commit that referenced this pull request Mar 31, 2026
(cherry picked from commit 5016060)
(cherry picked from commit f9e03d6)
michaelklishin added a commit that referenced this pull request Mar 31, 2026
OAuth 2 management UI: adopt a better approach than #15148 (backport #15887) (backport #15888)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants