Merged PR 799: makepasswordで条件に合致しないパスワードを生成した際無限ループになる
## 概要 [Task3840: makepasswordで条件に合致しないパスワードを生成した際無限ループになる](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/3840) - makepasswordで条件に合致しないパスワードを生成した場合、再度生成するループ処理としているが、初期化が行われていないため常に同じパスワードで条件合致のチェックが行われていました。結果、一度条件に合致しないパスワードを生成した場合無限ループとなっていました。 - ループ内で変数を初期化するよう対応。 ## レビューポイント - 特にありません ## UIの変更 - 無し ## 動作確認状況 - ユニットテストが通ることを確認 - ローカルでユーザー作成を実施し、これまで同様作成できることを確認 ## 補足 - 相談、参考資料などがあれば
This commit is contained in:
parent
a47ebaa9df
commit
2220e2560f
@ -15,9 +15,12 @@ export const makePassword = (): string => {
|
||||
|
||||
// autoGeneratedPasswordが以上の条件を満たせばvalidがtrueになる
|
||||
let valid = false;
|
||||
let autoGeneratedPassword: string = '';
|
||||
let autoGeneratedPassword = '';
|
||||
|
||||
while (!valid) {
|
||||
// 再生成用に変数を初期化する
|
||||
autoGeneratedPassword = '';
|
||||
|
||||
// パスワードをランダムに決定
|
||||
while (autoGeneratedPassword.length < passLength) {
|
||||
// 上で決定したcharsの中からランダムに1文字ずつ追加
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user