scottk/vault
1
0
Fork 0
mirror of https://github.com/optim-enterprises-bv/vault.git synced 2025-10-30 02:02:43 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix OktaNumberChallenge (#23565)

Browse Source 
* remove arg

* changelog
This commit is contained in:
Angel Garbarino
2023-10-11 09:12:44 -06:00
committed by GitHub
parent 83ae451df8
commit f144d1be54
2 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
changelog/23565.txt Normal file
View File

@@ -0,0 +1,3 @@
```release-note:bug
ui: Fix regression that broke the oktaNumberChallenge on the ui.
```

4
ui/app/components/auth-form.js
View File

@@ -269,7 +269,7 @@ export default Component.extend(DEFAULTS, {
return; return;
} }
let response = null; let response = null;
this.args.setOktaNumberChallenge(true); this.setOktaNumberChallenge(true);
this.setCancellingAuth(false); this.setCancellingAuth(false);
// keep polling /auth/okta/verify/:nonce API every 1s until a response is given with the correct number for the Okta Number Challenge // keep polling /auth/okta/verify/:nonce API every 1s until a response is given with the correct number for the Okta Number Challenge
while (response === null) { while (response === null) {
@@ -332,7 +332,7 @@ export default Component.extend(DEFAULTS, {
}); });
}, },
returnToLoginFromOktaNumberChallenge() { returnToLoginFromOktaNumberChallenge() {
this.args.setOktaNumberChallenge(false); this.setOktaNumberChallenge(false);
this.set('oktaNumberChallengeAnswer', null); this.set('oktaNumberChallengeAnswer', null);
}, },
}, },