Forum Discussion
Dev channel update to 92.0.884.2 is live
rshupak oh and also, for the SSO issue, can you try clearing your cookies and seeing if that helps? If you're signed into Edge with your MSA, you're right that it's what should be used for SSO instead of your AAD account, which makes me think that maybe your AAD credentials are just saved and being used, and so getting rid of them may help.
- rshupakMay 14, 2021Iron Contributor
When AAD SSO is disabled, Edge sends the following, which I elided, on the first request to https://login.microsoftonline.com/common/oauth2/v2.0/authorize?client_id=09213cdc-9f30-4e82-aa6f-9b6e8d82dab3&redirect_uri=https%3A%2F%2Ftechcommunity.microsoft.com%2Fauth%2Foauth2callback&response_type=code&state=https%3A%2F%2Ftechcommunity.microsoft.com%2F&scope=User.Read+openid+email+profile+offline_access
Cookie: ESTSAUTHLIGHT=+00000000-0000-0000-0000-000000000000; ESTSSC=00; esctx=...; x-ms-gateway-slice=estsfd; stsservicecookie=estsfd; brcap=0; ESTSAUTHPERSISTENT=...; ESTSAUTH=...; ch=...; buid=...; fpc=...; clrc={...]}
When AAD SSO is enabled, the Cookie header is not sent but the following is
x-ms-RefreshTokenCredential: ey...n0.ey...d4
It seems that with one or the other, there is no way this can work well for sites that support both AAD AND MSA since they are not provided with both credentials in order to present the user with a choice. As implemented right now, I need to change the Edge setting off to allow signing in with the MSA at all since otherwise the AAD account is selected. Some sites, such as this one, don't even allow a choice after signing out. In contrast, Edge will select AAD with no user choice on https://portal.azure.com/ or https://www.office.com/ but after being signed in you can sign out and then are presented with a choice immediately (portal.azure.com) or for the next sign in (http://www.office.com/)
- rshupakMay 14, 2021Iron ContributorIt occurs on multiple devices and channels all of which are configured to clear all but a few cookies on exit and no login cookies are preserved. It is in part a problem with this site as I have had this problem with the old Edge both here and on a few other Microsoft sites. It can be done correctly since some sites work. It seems worse with the new Edge feature which is also broken on https://www.office.com and https://portal.azure.com. https://developer.microsoft.com/en-us/graph/graph-explorer is one site which defaults to the AAD account but prompts for account selection before signing in.
- josh_bodnerMay 22, 2021Former Employee
rshupak for the settings page crashing when you search, we've actually made a few fixes this week that should make their way into next week's Dev. A couple of those crashes were only on Mac, and I use Windows machines, so that's likely why I wasn't seeing them.
As for the SSO behavior, I think I may have initially misunderstood what your problem was. The setting to "Allow single sign-on for work or school sites using this profile" actually forces SSO using your AAD account when it's turned on since AAD is the same as a work/school account (and thus MSA equals personal). So, since it sounds like that setting is working as intended, I think the problem is maybe that this website support both kinds of logins, and thus the problem is with the site? If so, that's feedback you should definitely give to the site.
- rshupakMay 25, 2021Iron Contributor
josh_bodner I don't know if I didn't try this scenario or if it regressed but these settings are still broken in canary Version 92.0.899.0 (Official build) canary (64-bit).
First, when opening edge://settings/content still starts with an error
GET edge://resources/js/cr/ui/command.js net::ERR_FAILED
assert.js:85 crbug/1173575, non-JS module files deprecated.
(anonymous) @ assert.js:85
util.js:320 crbug/1173575, non-JS module files deprecated.
(anonymous) @ util.js:320
promise_resolver.js:84 crbug/1173575, non-JS module files deprecated.
(anonymous) @ promise_resolver.js:84
icon.js:159 crbug/1173575, non-JS module files deprecated.
(anonymous) @ icon.js:159
define @ cr.js:295
(anonymous) @ icon.js:7
ui.js:213 crbug/1173575, non-JS module files deprecated.
(anonymous) @ ui.js:213
define @ cr.js:295
(anonymous) @ ui.js:5
keyboard_shortcut_list.js:94 crbug/1173575, non-JS module files deprecated.
(anonymous) @ keyboard_shortcut_list.js:94
define @ cr.js:295
(anonymous) @ keyboard_shortcut_list.js:5
parse_html_subset.js:218 crbug/1173575, non-JS module files deprecated.
(anonymous) @ parse_html_subset.js:218
load_time_data.js:215 crbug/1173575, non-JS module files deprecated.
(anonymous) @ load_time_data.js:215This page has a Recent activity section which I believe was a factor in the previous bug that was fixed. In my case, this section lists three sites only one of which has had any recent change in permissions. The first, which hasn't changed recently is https://ntp.msn.com/
When I click on the > on the right, this opens edge://settings/content/siteDetails which curiously lacks any query string to indicate what site it is trying to display. Regardless, this fails with
lib_react.chunk.js:9 Error: Unexpected condition on edge://settings/content/siteDetails: Could not find value for siteSettingsSourceDrmDisabled
at expect (load_time_data.js:191)
at LoadTimeData.getValue (load_time_data.js:57)
at LoadTimeData.getString (load_time_data.js:67)
at YT.getSublabelText (settings.bundle.js:1)
at settings.bundle.js:1
at Array.map (<anonymous>)
at YT.renderSelectRows (settings.bundle.js:1)
at YT.render (settings.bundle.js:1)
at Ha (lib_react.chunk.js:9)
at Ua (lib_react.chunk.js:9)
at ku (lib_react.chunk.js:9)
at ml (lib_react.chunk.js:9)
at vl (lib_react.chunk.js:9)
at ul (lib_react.chunk.js:9)
at lib_react.chunk.js:9
at t.unstable_runWithPriority (lib_common.chunk.js:38)
at Bi (lib_react.chunk.js:9)
at Yi (lib_react.chunk.js:9)
at qi (lib_react.chunk.js:9)
at ll (lib_react.chunk.js:9)
at Object.notify (lib_react.chunk.js:17)
at e.t.notifyNestedSubs (lib_react.chunk.js:17)
at e.t.handleChangeWrapper (lib_react.chunk.js:17)
at g (lib_react.chunk.js:25)
at lib_react.chunk.js:25
at dispatch (lib_react.chunk.js:25)
at settings.bundle.js:1
iu @ lib_react.chunk.js:9
n.callback @ lib_react.chunk.js:9
go @ lib_react.chunk.js:9
su @ lib_react.chunk.js:9
Sl @ lib_react.chunk.js:9
t.unstable_runWithPriority @ lib_common.chunk.js:38
Bi @ lib_react.chunk.js:9
wl @ lib_react.chunk.js:9
ul @ lib_react.chunk.js:9
(anonymous) @ lib_react.chunk.js:9
t.unstable_runWithPriority @ lib_common.chunk.js:38
Bi @ lib_react.chunk.js:9
Yi @ lib_react.chunk.js:9
qi @ lib_react.chunk.js:9
ll @ lib_react.chunk.js:9
notify @ lib_react.chunk.js:17
t.notifyNestedSubs @ lib_react.chunk.js:17
t.handleChangeWrapper @ lib_react.chunk.js:17
g @ lib_react.chunk.js:25
(anonymous) @ lib_react.chunk.js:25
dispatch @ lib_react.chunk.js:25
(anonymous) @ settings.bundle.js:1
async function (async)
(anonymous) @ settings.bundle.js:1
(anonymous) @ settings.bundle.js:1
(anonymous) @ lib_react.chunk.js:25
(anonymous) @ settings.bundle.js:1
Promise.then (async)
componentWillMount @ settings.bundle.js:1
To @ lib_react.chunk.js:9
Ua @ lib_react.chunk.js:9
ku @ lib_react.chunk.js:9
ml @ lib_react.chunk.js:9
vl @ lib_react.chunk.js:9
ul @ lib_react.chunk.js:9
(anonymous) @ lib_react.chunk.js:9
t.unstable_runWithPriority @ lib_common.chunk.js:38
Bi @ lib_react.chunk.js:9
Yi @ lib_react.chunk.js:9
qi @ lib_react.chunk.js:9
j @ lib_react.chunk.js:9
Yt @ lib_react.chunk.js:9
settings.bundle.js:1 Uncaught (in promise) Error: Unexpected condition on edge://settings/content/siteDetails: Could not find value for siteSettingsSourceDrmDisabled
at expect (load_time_data.js:191)
at LoadTimeData.getValue (load_time_data.js:57)
at LoadTimeData.getString (load_time_data.js:67)
at YT.getSublabelText (settings.bundle.js:1)
at settings.bundle.js:1
at Array.map (<anonymous>)
at YT.renderSelectRows (settings.bundle.js:1)
at YT.render (settings.bundle.js:1)
at Ha (lib_react.chunk.js:9)
at Ua (lib_react.chunk.js:9)
at ku (lib_react.chunk.js:9)
at ml (lib_react.chunk.js:9)
at vl (lib_react.chunk.js:9)
at ul (lib_react.chunk.js:9)
at lib_react.chunk.js:9
at t.unstable_runWithPriority (lib_common.chunk.js:38)
at Bi (lib_react.chunk.js:9)
at Yi (lib_react.chunk.js:9)
at qi (lib_react.chunk.js:9)
at ll (lib_react.chunk.js:9)
at Object.notify (lib_react.chunk.js:17)
at e.t.notifyNestedSubs (lib_react.chunk.js:17)
at e.t.handleChangeWrapper (lib_react.chunk.js:17)
at g (lib_react.chunk.js:25)
at lib_react.chunk.js:25
at dispatch (lib_react.chunk.js:25)
at settings.bundle.js:1
expect @ load_time_data.js:191
getValue @ load_time_data.js:57
getString @ load_time_data.js:67
(anonymous) @ settings.bundle.js:1
(anonymous) @ settings.bundle.js:1
(anonymous) @ settings.bundle.js:1
render @ settings.bundle.js:1
Ha @ lib_react.chunk.js:9
Ua @ lib_react.chunk.js:9
ku @ lib_react.chunk.js:9
ml @ lib_react.chunk.js:9
vl @ lib_react.chunk.js:9
ul @ lib_react.chunk.js:9
(anonymous) @ lib_react.chunk.js:9
t.unstable_runWithPriority @ lib_common.chunk.js:38
Bi @ lib_react.chunk.js:9
Yi @ lib_react.chunk.js:9
qi @ lib_react.chunk.js:9
ll @ lib_react.chunk.js:9
notify @ lib_react.chunk.js:17
t.notifyNestedSubs @ lib_react.chunk.js:17
t.handleChangeWrapper @ lib_react.chunk.js:17
g @ lib_react.chunk.js:25
(anonymous) @ lib_react.chunk.js:25
dispatch @ lib_react.chunk.js:25
(anonymous) @ settings.bundle.js:1
async function (async)
(anonymous) @ settings.bundle.js:1
(anonymous) @ settings.bundle.js:1
(anonymous) @ lib_react.chunk.js:25
(anonymous) @ settings.bundle.js:1
Promise.then (async)
componentWillMount @ settings.bundle.js:1
To @ lib_react.chunk.js:9
Ua @ lib_react.chunk.js:9
ku @ lib_react.chunk.js:9
ml @ lib_react.chunk.js:9
vl @ lib_react.chunk.js:9
ul @ lib_react.chunk.js:9
(anonymous) @ lib_react.chunk.js:9
t.unstable_runWithPriority @ lib_common.chunk.js:38
Bi @ lib_react.chunk.js:9
Yi @ lib_react.chunk.js:9
qi @ lib_react.chunk.js:9
j @ lib_react.chunk.js:9
Yt @ lib_react.chunk.js:9