Skip to content

fix: #259 The HTTP Authentication Scheme patterns should be case-inse…#260

Open
AlwinGarside wants to merge 1 commit intothephpleague:masterfrom
AlwinGarside:bugfix/Security-Auth-Pattern-should-not-be-case-sensitive
Open

fix: #259 The HTTP Authentication Scheme patterns should be case-inse…#260
AlwinGarside wants to merge 1 commit intothephpleague:masterfrom
AlwinGarside:bugfix/Security-Auth-Pattern-should-not-be-case-sensitive

Conversation

@AlwinGarside
Copy link
Copy Markdown

We were running into random "None of security schemas did match for …" validation errors. This was being caused by the League\OpenAPIValidation\PSR7\Validators\SecurityValidator:: AUTH_PATTERN_* patterns, which do not have the case-insensitivity flag (i) set.

The basic and bearer authentication scheme tokens should be considered case-insensitive per Section 2.1 of RFC 7235. So this PR adds the i flag so that the matching is done in a case-insensitive manner.

…be case-insensitive

The `basic` and `bearer` authentication scheme
tokens should be considered case-insensitive per
[Section 2.1 of RFC 7235](https://datatracker.ietf.org/doc/html/rfc7235#section-2.1).
@AlwinGarside
Copy link
Copy Markdown
Author

@scaytrase Any chance this could be merged? 👀

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