Skip to content

Conversation

jonnekleijer
Copy link
Contributor

Fixes #844: The regex ignores numbers in custom auth providers. Although various documented providers have numbers in their description (e.g. auth0, aadb2c). As a consequence they will not work with the local authorization emulator.

various documented providers currently do not work with the local authorization emulator (e.g. auth0, aadb2c), because numbers are not captured in the regex

Fixes Azure#844
various documented providers currently do not work with the local authorization emulator (e.g. auth0, aadb2c), because numbers are not captured in the regex

Fixes Azure#844
@github-actions github-actions bot added the scope: auth Issues related to the authentication emulator label Nov 5, 2024
Copy link
Member

@Timothyw0 Timothyw0 left a comment

Choose a reason for hiding this comment

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

Thank you for catching this!

@Timothyw0 Timothyw0 merged commit c74de88 into Azure:main Nov 27, 2024
2 checks passed
Timothyw0 pushed a commit to Timothyw0/static-web-apps-cli that referenced this pull request Dec 15, 2024
…ty-provider-regex"

This reverts commit c74de88, reversing
changes made to 6745391.
Timothyw0 pushed a commit to Timothyw0/static-web-apps-cli that referenced this pull request Dec 27, 2024
jstoker added a commit to jstoker/static-web-apps-cli that referenced this pull request Mar 6, 2025
…bers in their names

Fixes regression from Azure#844 within the 2.x release and applies the same partial fix in Azure#895 in the other code path.
jstoker added a commit to jstoker/static-web-apps-cli that referenced this pull request Mar 6, 2025
…ders with numbers

Fix regression from Azure#844 within the 2.x release and applies the same partial fix in Azure#895 in the other code path.
This is to support providers such as Azure AD B2C or Auth0, which usually would be using 'aadb2c' or 'auth0'.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope: auth Issues related to the authentication emulator
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Custom OpenIdConnectProviders is broken in 1.1.8
2 participants