Merge pull request #20655 from TheGemmell/master
Patch for microsoft-provider prompt bug
This commit is contained in:
@@ -0,0 +1,5 @@
|
||||
---
|
||||
'@backstage/plugin-auth-backend-module-microsoft-provider': patch
|
||||
---
|
||||
|
||||
Removed `prompt=consent` from start method to fix #20641
|
||||
@@ -60,7 +60,6 @@ export const microsoftAuthenticator = createOAuthAuthenticator({
|
||||
async start(input, helper) {
|
||||
return helper.start(input, {
|
||||
accessType: 'offline',
|
||||
prompt: 'consent',
|
||||
});
|
||||
},
|
||||
|
||||
|
||||
@@ -65,7 +65,6 @@ describe('authModuleMicrosoftProvider', () => {
|
||||
expect(startUrl.origin).toBe('https://login.microsoftonline.com');
|
||||
expect(startUrl.pathname).toBe('/my-tenant-id/oauth2/v2.0/authorize');
|
||||
expect(Object.fromEntries(startUrl.searchParams)).toEqual({
|
||||
prompt: 'consent',
|
||||
response_type: 'code',
|
||||
scope: 'user.read',
|
||||
client_id: 'my-client-id',
|
||||
|
||||
Reference in New Issue
Block a user