From 97bb9c4190e2dbc835228b35802c7acd5d291952 Mon Sep 17 00:00:00 2001 From: Kentaro Fukunaga Date: Thu, 22 Jun 2023 06:28:57 +0000 Subject: [PATCH] =?UTF-8?q?Merged=20PR=20174:=20=E3=82=A2=E3=82=AB?= =?UTF-8?q?=E3=82=A6=E3=83=B3=E3=83=88=E7=99=BB=E9=8C=B2=E6=99=82=E3=83=BB?= =?UTF-8?q?=E3=83=A6=E3=83=BC=E3=82=B6=E3=83=BC=E8=BF=BD=E5=8A=A0=E6=99=82?= =?UTF-8?q?=E3=81=AE=E3=83=91=E3=82=B9=E3=83=AF=E3=83=BC=E3=83=89=E3=83=9D?= =?UTF-8?q?=E3=83=AA=E3=82=B7=E3=83=BC=E3=82=92=E3=80=81AADB2C=E6=A8=99?= =?UTF-8?q?=E6=BA=96=E3=81=AB=E5=90=88=E3=82=8F=E3=81=9B=E3=82=8B?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ## 概要 [Task1958: アカウント登録時・ユーザー追加時のパスワードポリシーを、AADB2C標準に合わせる](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1958) - 利用可能な文字種を修正 - 開発規約上「推奨」となっていたルールを排除 - フロント側の潜在バグ（パスワードの長さチェック）を修正かつリファクタ ## レビューポイント - 無駄な書き方になっているようなところはないか ## UIの変更 - 無し ## 動作確認状況 - ローカルでアカウント登録時、パスワードポリシー満たしていないものであれば次画面遷移できないことを確認 - ADB2Cユーザーフロー画面でパスワードリセット時に通過するルールと一致していることを確認しました。 ## 補足 - 相談、参考資料などがあれば --- .../src/features/signup/selectors.ts | 30 ++++++++----------- .../src/common/password/password.ts | 17 +++-------- 2 files changed, 17 insertions(+), 30 deletions(-) diff --git a/dictation_client/src/features/signup/selectors.ts b/dictation_client/src/features/signup/selectors.ts index 4d74956..a6625d3 100644 --- a/dictation_client/src/features/signup/selectors.ts +++ b/dictation_client/src/features/signup/selectors.ts @@ -11,7 +11,7 @@ export const selectInputValidationErrors = (state: RootState) => { const hasErrorEmptyDealer = state.signup.apps.dealer === ""; */ - const hasErrorIncorrectPassword = checkErrorIncorrectPassword( + const hasErrorIncorrectPassword = !isValidPasswordFormat( state.signup.apps.password ); @@ -28,23 +28,19 @@ export const selectInputValidationErrors = (state: RootState) => { hasErrorIncorrectPassword, }; }; -export const checkErrorIncorrectPassword = (password: string): boolean => { - // 英字の大文字、英字の小文字、アラビア数字、記号（!@#$%^&*()+-={}[]:;'<>,./?_∼\）から2種類以上組み合わせ + +// 渡された文字列がパスワードポリシーに一致しているかを判定する +const isValidPasswordFormat = (password: string): boolean => { + // 8文字～64文字でなければ早期に不合格 + const minLength = 8; + const maxLength = 64; + if (password.length < minLength || password.length > maxLength) return false; + + // 英字の大文字、英字の小文字、アラビア数字、記号（@#$%^&*\-_+=[]{}|\:',.?/`~"();!）から2種類以上組み合わせ const charaTypePattern = - /^((?=.*[a-z])(?=.*[A-Z])|(?=.*[a-z])(?=.*[\d])|(?=.*[a-z])(?=.*[!@#$%^&*()+-={}:;'<>,./?_~[\\\]])|(?=.*[A-Z])(?=.*[\d])|(?=.*[A-Z])(?=.*[!@#$%^&*()+-={}:;'<>,./?_~[\\\]])|(?=.*[\d])(?=.*[!@#$%^&*()+-={}:;'<>,./?_~[\\\]]))[a-zA-Z\d!@#$%^&*()+-={}:;'<>,./?_~[\\\]]{8,64}$/; - const charaType = new RegExp(charaTypePattern).test(password); - - // 同じ文字の3連続は禁止 - const repeatPattern = /(.)\1{2,}/; - const repeat = new RegExp(repeatPattern).test(password); - - // 特定文字列は禁止 - const unavailableCharaPattern = - /password|passwd|test|admin|administrator|sysadmin|0123|1234|2345|3456|4567|5678|6789|9876|8765|7654|6543|5432|4321|3210/; - const unavailableChara = !new RegExp(unavailableCharaPattern, "i").test( - password - ); - return !charaType || repeat || !unavailableChara; + // eslint-disable-next-line no-useless-escape + /^((?=.*[a-z])(?=.*[A-Z])|(?=.*[a-z])(?=.*[\d])|(?=.*[a-z])(?=.*[@#$%^&*\\\-_+=\[\]{}|:',.?\/`~"();!])|(?=.*[A-Z])(?=.*[\d])|(?=.*[A-Z])(?=.*[@#$%^&*\\\-_+=\[\]{}|:',.?\/`~"();!])|(?=.*[\d])(?=.*[@#$%^&*\\\-_+=\[\]{}|:',.?\/`~"();!]))[a-zA-Z\d@#$%^&*\\\-_+=\[\]{}|:',.?\/`~"();!]/; + return new RegExp(charaTypePattern).test(password); }; // Account Info diff --git a/dictation_server/src/common/password/password.ts b/dictation_server/src/common/password/password.ts index b54580f..76265e2 100644 --- a/dictation_server/src/common/password/password.ts +++ b/dictation_server/src/common/password/password.ts @@ -6,19 +6,12 @@ export const makePassword = (): string => { const lowerCase = 'abcdefghijklmnopqrstuvwxyz'; const upperCase = lowerCase.toLocaleUpperCase(); const numbers = '0123456789'; - const symbols = "!@#$%^&*()+-={}[]:;'<>,./?_∼\\"; + const symbols = '@#$%^&*\\-_+=[]{}|:\',.?/`~"();!'; const chars = lowerCase + upperCase + numbers + symbols; - // 英字の大文字、英字の小文字、アラビア数字、記号（!@#$%^&*()+-={}[]:;'<>,./?_∼\）から2種類以上組み合わせ + // 英字の大文字、英字の小文字、アラビア数字、記号（@#$%^&*\-_+=[]{}|\:',.?/`~"();!）から2種類以上組み合わせ const charaTypePattern = - /^((?=.*[a-z])(?=.*[A-Z])|(?=.*[a-z])(?=.*[\d])|(?=.*[a-z])(?=.*[!@#$%^&*()+-={}:;'<>,./?_~[\\\]])|(?=.*[A-Z])(?=.*[\d])|(?=.*[A-Z])(?=.*[!@#$%^&*()+-={}:;'<>,./?_~[\\\]])|(?=.*[\d])(?=.*[!@#$%^&*()+-={}:;'<>,./?_~[\\\]]))[a-zA-Z\d!@#$%^&*()+-={}:;'<>,./?_~[\\\]]/; - - // 同じ文字の3連続は禁止 - const repeatPattern = /(.)\1{2,}/; - - // 特定文字列は禁止 - const unavailableCharaPattern = - /password|passwd|test|admin|administrator|sysadmin|0123|1234|2345|3456|4567|5678|6789|9876|8765|7654|6543|5432|4321|3210/; + /^((?=.*[a-z])(?=.*[A-Z])|(?=.*[a-z])(?=.*[\d])|(?=.*[a-z])(?=.*[@#$%^&*\\\-_+=\[\]{}|:',.?\/`~"();!])|(?=.*[A-Z])(?=.*[\d])|(?=.*[A-Z])(?=.*[@#$%^&*\\\-_+=\[\]{}|:',.?\/`~"();!])|(?=.*[\d])(?=.*[@#$%^&*\\\-_+=\[\]{}|:',.?\/`~"();!]))[a-zA-Z\d@#$%^&*\\\-_+=\[\]{}|:',.?\/`~"();!]/; // autoGeneratedPasswordが以上の条件を満たせばvalidがtrueになる let valid = false; @@ -37,9 +30,7 @@ export const makePassword = (): string => { // 条件を満たすまでループ valid = autoGeneratedPassword.length == passLength && - charaTypePattern.test(autoGeneratedPassword) && - !repeatPattern.test(autoGeneratedPassword) && - !unavailableCharaPattern.test(autoGeneratedPassword); + charaTypePattern.test(autoGeneratedPassword); } return autoGeneratedPassword; };