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

[Bug] Redirecting to microsoft login #4525

Open
Neena-Mohan opened this issue Nov 18, 2024 · 3 comments
Open

[Bug] Redirecting to microsoft login #4525

Neena-Mohan opened this issue Nov 18, 2024 · 3 comments
Assignees
Labels
Awaiting Reproduction Can we reproduce the reported bug?

Comments

@Neena-Mohan
Copy link

Neena-Mohan commented Nov 18, 2024

Describe the Bug

We're using Azure dicom service for OHIF Viewer. In that, we gave azure login(authority) in config file for authentication. When loading the ohif viewer, it redirecting to microsoft login at first. For client, It should not redirect to there at first time. So please provide a solution for this.

oidc: [
{
authority: "https://login.microsoftonline.com/****/v2.0/",
client_id: "CLIENT_ID",
redirect_uri: "/callback",
response_type: "token",
scope: "openid ****",
post_logout_redirect_uri: "/logout-redirect.html",
automaticSilentRenew: false,
revokeAccessTokenOnSignout: true,
},
],

Steps to Reproduce

Loading the url in browser
Sample URL : https://domainname.z13.web.core.windows.net/viewer?StudyInstanceUIDs=***&token=***

At first, if the user microsoft account is not logged-in then it redirect to microsoft login.

The current behavior

Redirecting to microsoft login when loading the url at first time. If my microsoft account is already logged then its not going login page.

The expected behavior

It should not redirect to login page

OS

Windows 10

Node version

Using bundle files

Browser

Chrome 130.0.6723.117

@Neena-Mohan Neena-Mohan added the Awaiting Reproduction Can we reproduce the reported bug? label Nov 18, 2024
Copy link
Member

sedghi commented Nov 18, 2024

We plan to add some doc for Azure auth soon

@Neena-Mohan
Copy link
Author

@sedghi @IbrahimCSAE Could you help me with this? Is there anything we need to adjust in the configuration to resolve this issue? This is urgent, as we've halted production due to it.

@sedghi
Copy link
Member

sedghi commented Nov 18, 2024

#4525 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Reproduction Can we reproduce the reported bug?
Projects
None yet
Development

No branches or pull requests

3 participants