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

[Feature] Add AAD Custom Auth Support & Setup Custom Auth Framework #878

Merged
merged 9 commits into from
Aug 13, 2024

Conversation

Timothyw0
Copy link
Member

@Timothyw0 Timothyw0 commented Aug 7, 2024

New feature: Adding AAD custom auth support using the previously created code for Google and GitHub. The previous code has been refactored and generalized for future auth providers to be added as well. Google and Github auth has also been regression tested successfully.

Tested in private environment with custom AAD application using the following staticwebapp.config.json:
image

Successful /.auth/me call with AAD credentials:
image

Successful /.auth/me call with Github credentials:
image

Successful /.auth/me call with Google credentials:
image

@Timothyw0 Timothyw0 requested a review from cjk7989 as a code owner August 7, 2024 17:55
@Timothyw0 Timothyw0 requested review from andimarc and adrianhall and removed request for cjk7989 August 7, 2024 17:55
@github-actions github-actions bot added scope: core Issues happened a the ./src/core level scope: auth Issues related to the authentication emulator labels Aug 7, 2024
@Timothyw0 Timothyw0 requested a review from mkarmark August 7, 2024 19:51
@Timothyw0 Timothyw0 changed the title [Feature] Add AAD Custom Auth Support [Feature] Add AAD Custom Auth Support & Setup Custom Auth Framework Aug 7, 2024
src/core/constants.ts Outdated Show resolved Hide resolved
src/msha/auth/index.ts Outdated Show resolved Hide resolved
src/msha/auth/index.ts Show resolved Hide resolved
…tching to use supported auth constant list.
@Timothyw0 Timothyw0 requested a review from cjk7989 August 13, 2024 17:43
@Timothyw0 Timothyw0 merged commit 7938384 into Azure:main Aug 13, 2024
18 checks passed
Timothyw0 pushed a commit to Timothyw0/static-web-apps-cli that referenced this pull request Dec 15, 2024
This reverts commit 7938384, reversing
changes made to 8d0b7aa.
Timothyw0 pushed a commit to Timothyw0/static-web-apps-cli that referenced this pull request Dec 27, 2024
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 scope: core Issues happened a the ./src/core level
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants