-
Notifications
You must be signed in to change notification settings - Fork 23
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
Popup doesn't close if the history mode is not on #4
Comments
Which browser is it happening in? This was a common issue with AngularJS I
think. Have you checked stackoverflow?
…On Wed, May 16, 2018, 11:13 PM baykingsea ***@***.***> wrote:
Sunil - thanks for building a great example. I had tried some other vue
npms and samples nothing worked except yours for MSAL. I hope this question
is for real since the last two from me turned out to have answers in the
services
The code works fine. But when I move the code to production which doesn't
support the history mode i.e. my production url has a # in it, it doesn't
close the popup. I can see that authorization has happened and I can see
tokens in popup window url has well. I unfortunately cannot turn off the #
in the production url because that is how the client wants it. Is there a
way to handle this scenario?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#4>, or mute the
thread
<https://github.com/notifications/unsubscribe-auth/AAqLnZ-8ZtdgCC9x-0GJUroPPfYTLxgVks5tzRUegaJpZM4UCd39>
.
|
Both Chrome and Firefox. I couldn't find anything on the stack overflow else I would not have troubled you. Thanks again for creating this plugin and your help. |
The homepage url looks like this: www.mydomain.com/website#/ If the has is not there it works fine then. |
Have you already created an issue in the msal repo with the repro code?
…On Thu, May 17, 2018, 8:24 PM baykingsea ***@***.***> wrote:
The homepage url looks like this: www.mydomain.com/website#/
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#4 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAqLnd9qK6MKqx_ghURFpdprE6gXfRGMks5tzj7ZgaJpZM4UCd39>
.
|
No I haven't. I am using your code, without any changes. |
Can you please check with them if such URLs are supported?
…On Thu, May 17, 2018, 9:53 PM baykingsea ***@***.***> wrote:
No I haven't. I am using your code, without any changes.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#4 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAqLnRpzf4BR5tNCu0Z3MHJeng68UtYeks5tzlPBgaJpZM4UCd39>
.
|
Hey @baykingsea , I came across this issue and figured out why. I don't think it has to do with the hash, but I could be wrong. Mine was doing same thing but I don't have hashes in url. I was instantiating the msal library in a component that was within a Just thought I'd share... Nice example @sunilbandla |
Can you try creating a new application in the portal and add the reply URL? If that doesn't work, asking on stackoverflow will help. |
Sunil - thanks for building a great example. I had tried some other vue npms and samples nothing worked except yours for MSAL. I hope this question is for real since the last two from me turned out to have answers in the services
The code works fine. But when I move the code to production which doesn't support the history mode i.e. my production url has a # in it, it doesn't close the popup. I can see that authorization has happened and I can see tokens in popup window url has well. I unfortunately cannot turn off the # in the production url because that is how the client wants it. Is there a way to handle this scenario?
The text was updated successfully, but these errors were encountered: