Skip to content

AAD Custom Authentication emulation settings are not compatible with the CLI #928

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

Open
1 of 2 tasks
shibayan opened this issue Feb 14, 2025 · 1 comment · May be fixed by #933
Open
1 of 2 tasks

AAD Custom Authentication emulation settings are not compatible with the CLI #928

shibayan opened this issue Feb 14, 2025 · 1 comment · May be fixed by #933

Comments

@shibayan
Copy link
Contributor

Before filing this issue, please ensure you're using the latest CLI by running swa --version and comparing to the latest version on npm.

Are you accessing the CLI from the default port :4280 ?

  • No, I am using a different port number (--port) and accessing the CLI from that port
  • Yes, I am accessing the CLI from port :4280

Make sure you are accessing the URL printed in the console when running swa start!

ℹ️ NOTE: Make sure to enable debug logs when running any swa commands using --verbose=silly

Describe the bug

When using Entra ID (AAD) for SWA custom authentication, you must specify the format https://login.microsoftonline.com/<TENANT_ID>/v2.0 as openIdIssuer.

Image

However, as far as I have tested, the SWA CLI emulation does not work correctly unless you specify the format https://login.microsoftonline.com/<TENANT_ID>/oauth2/v2.0.

#878

I think it is necessary to make the format of the openIdIssuer accepted by the SWA CLI consistent with that of SWA Cloud.

To Reproduce
Steps to reproduce the behavior:

  1. Go to '...'
  2. Type in command 'swa ...'
  3. Click on '...'
  4. Scroll down to '...'
  5. See error

Expected behavior
A clear and concise description of what you expected to happen.

Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

  • OS: [e.g. iOS]
  • Version [e.g. 22]

Additional context
Add any other context about the problem here.

@ciacco85
Copy link

ciacco85 commented Apr 8, 2025

Related to #941

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 a pull request may close this issue.

2 participants