Skip to content

Commit

Permalink
remove migration parameters
Browse files Browse the repository at this point in the history
  • Loading branch information
thewahome committed Apr 24, 2024
1 parent ea8e5e5 commit b809067
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 16 deletions.
1 change: 0 additions & 1 deletion .github/workflows/linter.yml
Original file line number Diff line number Diff line change
Expand Up @@ -185,7 +185,6 @@ jobs:
REACT_APP_NOMINATION_PERIOD: ${{secrets.REACT_APP_NOMINATION_PERIOD}}
REACT_APP_COOLDOWN_PERIOD: ${{secrets.REACT_APP_COOLDOWN_PERIOD}}
REACT_APP_USAGE_TIME: ${{secrets.REACT_APP_USAGE_TIME}}
REACT_APP_MIGRATION_PARAMETER: ${{secrets.REACT_APP_MIGRATION_PARAMETER}}
CI: false
id: builddeploy
uses: Azure/static-web-apps-deploy@v0.0.1-preview
Expand Down
17 changes: 2 additions & 15 deletions src/modules/authentication/AuthenticationWrapper.ts
Original file line number Diff line number Diff line change
Expand Up @@ -32,18 +32,6 @@ export class AuthenticationWrapper implements IAuthenticationWrapper {
selectedVersion: '',
sampleHeaders: []
};
private extraQueryParameters: { [key: string]: string } = (() => {
const params: { [key: string]: string } = {
mkt: geLocale
};

const migrationParam = process.env.REACT_APP_MIGRATION_PARAMETER;
if (migrationParam) {
params.safe_rollout = migrationParam;
}

return params;
})();

public static getInstance(): AuthenticationWrapper {
if (!AuthenticationWrapper.instance) {
Expand Down Expand Up @@ -85,7 +73,7 @@ export class AuthenticationWrapper implements IAuthenticationWrapper {
authority: this.getAuthority(),
prompt: 'select_account',
redirectUri: getCurrentUri(),
extraQueryParameters: this.extraQueryParameters
extraQueryParameters: { mkt: geLocale }
};
try {
const result = await msalApplication.loginPopup(popUpRequest);
Expand Down Expand Up @@ -217,10 +205,9 @@ export class AuthenticationWrapper implements IAuthenticationWrapper {
authority: this.getAuthority(),
prompt: 'select_account',
redirectUri: getCurrentUri(),
extraQueryParameters: this.extraQueryParameters,
extraQueryParameters: { mkt: geLocale },
claims: this.getClaims()
};
console.log('popUpRequest', popUpRequest)

if (this.consentingToNewScopes || this.performingStepUpAuth) {
delete popUpRequest.prompt;
Expand Down

0 comments on commit b809067

Please sign in to comment.