Merged PR 685: authページでinstanceの取得に失敗した場合でもログインページに遷移させる
## 概要 [Task3476: authページでinstanceの取得に失敗した場合でもログインページに遷移させる](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3476) 以下暫定対応です。 `const loginResult = await instance.handleRedirectPromise();` 推測としては、この処理でloginResultがnullになってしまうことがあり、 結果、ページ遷移などが発生しないままになってまうため、loading表示で固まる。 そのため、いかなる場合でも最終的にはログインページに遷移させるよう修正した。 これによって、loginResultがnullの場合は、ログインページに遷移後トップページに遷移し、ユーザは再度ログイン処理を行うことができる。 一旦この状態でテストは継続していただいて、根本原因調査は裏で行う。 ## レビューポイント - console.log({ loginResult }); はDEV環境での動作確認後に削除いたします。 ## UIの変更 なし ## 動作確認状況 - ローカルで確認 ## 補足 - 相談、参考資料などがあれば
This commit is contained in:
parent
d627bb8a09
commit
4c527b1959
@ -66,12 +66,10 @@ const AuthPage: React.FC = (): JSX.Element => {
|
||||
clearToken();
|
||||
return;
|
||||
}
|
||||
|
||||
const loginResult = await instance.handleRedirectPromise();
|
||||
|
||||
// eslint-disable-next-line
|
||||
console.log({ loginResult }); // TODO:loading画面から遷移できない事象の調査用ログ。事象解消後削除(eslint-disable含めて)する。
|
||||
|
||||
if (loginResult && loginResult.account) {
|
||||
const { homeAccountId, idTokenClaims } = loginResult.account;
|
||||
if (idTokenClaims && idTokenClaims.aud) {
|
||||
@ -85,11 +83,11 @@ const AuthPage: React.FC = (): JSX.Element => {
|
||||
localStorageKeyforIdToken,
|
||||
})
|
||||
);
|
||||
|
||||
// トークン取得と設定を行う
|
||||
navigate("/login");
|
||||
}
|
||||
}
|
||||
// ログインページに遷移し、トークン取得と設定を行う
|
||||
// 何らかの原因で、loginResultがnullの場合でも、ログイン画面に遷移する(ログイン画面でトップページに戻る)
|
||||
navigate("/login");
|
||||
} catch (e) {
|
||||
// eslint-disable-next-line
|
||||
console.log({ e }); // TODO:loading画面から遷移できない事象の調査用ログ。事象解消後削除(eslint-disable含めて)する。
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user