AzureAD / azure-activedirectory-library-for-js

The code for ADAL.js and ADAL Angular has been moved to the MSAL.js repo. Please open any issues or PRs at the link below.
https://github.com/AzureAD/microsoft-authentication-library-for-js/tree/dev/maintenance/adal-angular
Apache License 2.0
627 stars 374 forks source link

Stay signed in option not showing #830

Closed rafaelgrilli92 closed 4 years ago

rafaelgrilli92 commented 5 years ago

Library Name

adal.js

Library version

1.0.17

Current behavior

I had a nodejs server serving the client side (React) only to deal with the authentication, but then I took the nodejs out and I started to use this library on the client side instead. The thing is... the "stay signed in" option isn't showing anymore. I set it up on the azure portal to allow it, but it's still not working.

Any recommendations?

Expected behavior

image

jhill-dci commented 5 years ago

@rafaelgrilli92 I'm having the exact opposite issue - I want to suppress this message as users are authenticating via SSO integration, so this message is redundant. You mentioned you configured your Azure portal to allow this - would you mind elaborating?

rafaelgrilli92 commented 5 years ago

@jhill-dci, yea sure!

On the Azure portal, go to Active Directory > Company Branding > Click on "New Language"

Then on the bottom, you will find this: image

The thing is: All other options of this configuration are working for me, like username hint, sign in page text, etc... Except the remain signed in option =(

jhill-dci commented 5 years ago

That was fast - thank you! I'll keep digging to see if I can find a per-azure-app option to control this (wouldn't want to disable this company wide, but just on this particular authentication request).

Pheil commented 5 years ago

@jhill-dci Did you ever find a way to enable/disable per app?

jhill-dci commented 5 years ago

I did not, unfortunately. The issue comes up rarely so we're just dealing with it now (other projects bumped the priority of a fix here), but I'll circle back if and when we find a solution.

jmckennon commented 4 years ago

@sameerag is this fixed in msal@1.2.0?

jasonnutter commented 4 years ago

Closing due to inactivity.

rafaelgrilli92 commented 4 years ago

2 years later and it's still not showing up the option and the issue is being closed due to "inactivity". Noice!