Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

レビュー指摘修正版 #22

Merged
merged 78 commits into from
Mar 15, 2023
Merged

レビュー指摘修正版 #22

merged 78 commits into from
Mar 15, 2023

Conversation

sai-gillingham
Copy link
Contributor

PR3/4/10のレビュー指摘を修正しました。

kentanakayrglm and others added 30 commits January 15, 2023 20:32
@kentanakayrglm
・不要ロジックの削除
06048b6 
・Cookie有効期間の見直し
@KenTanaka
Merge pull request #2 from KenTanaka/develop/refactor1206
a60ffaa 
Develop/refactor1206
@kentanakayrglm
・PR対応（不要な処理・変数・useの削除）
c60e5af 
・管理画面のバリデーション不具合修正
・スロットリングの設定追加
@kentanakayrglm
- デフォルトログイン認証ルートの設定不具合修正
ffed27c
@sai-gillingham
Merge pull request #1 from KenTanaka/main
55b37c1 
修正後のバージョンを取得する
@sai-gillingham
デバイス認証済み電話番号をnullにしたら、ECCUBEシステムがバグする。
68effb7
@sai-gillingham
https://github.com/EC-CUBE/TwoFactorAuthCustomer42/issues/5
680ae20 
ログアウト時に2段階認証画面にアクセスするとシステムエラー
の修正。
@sai-gillingham
ゲスト登録不具合の修正。
43fd558
@sai-gillingham
本人認証が無効設定でも実行されるの修正
dc53ead 
本人認証と二段階認証のイベントリスナーを分割すること。
@sai-gillingham
本人認証が無効設定でも実行されるの修正
e247d14 
本人認証と二段階認証のイベントリスナーを分割すること。
EC-CUBE#7
@sai-gillingham
Merge remote-tracking branch 'origin/main'
652da9a
@sai-gillingham
会員編集画面の2段階認証方式項目に未設定オプションを追加する。
c942b7a 
EC-CUBE#11
@sai-gillingham
仮会員設定：無効
3bde284 
本人認証：無効
二段階認証：有効
パターンの対応。
EC-CUBE#7
@sai-gillingham
購入フローで二段階認証をスキップさせないように修正。
c93099b 
EC-CUBE#6
@sai-gillingham
APP認証プラグインを無効化した状態で、二段階認証にAPPが表示されないように修正
d12e1d4 
EC-CUBE#12
@sai-gillingham
無効になっているTwoFactorAuthTypeの行を消すことより無効になっているTwoFactorAuthType行を無効に設定する
154771d 
EC-CUBE#12
@kentanakayrglm
マイページ配下のページを重要操作ルーティングに指定しても、再認証されていない不具合を修正
c2ec8c8 
EC-CUBE#14
@sai-gillingham
２段階認証選択画面の選択肢修正。
997ad73 
EC-CUBE#12
@sai-gillingham
Merge remote-tracking branch 'origin/main'
4ba89a5
@sai-gillingham
２段階認証を利用する項目は不要になったため、two_factor_auth項目を消す。
203a6cc 
EC-CUBE#12
@sai-gillingham
２段階認証を利用する項目は不要になったため、two_factor_auth項目を消す。
53c3832 
EC-CUBE#12
@sai-gillingham
ワンタイムパスワードの値をハッシュ化にする。+ SMSの電話番号チェックの修正
220ae00 
EC-CUBE#13
@sai-gillingham
ログアウトするときに、二段階認証クッキーが消えるようになった。
2533067 
EC-CUBE#15
@sai-gillingham
デフォルト操作ルートのみで２段階認証クッキーデータをデータベースに保存する
ef8d2e3 
EC-CUBE#17
@sai-gillingham
ハッシュしているため、device_auth_one_time_token文字列のlengthを10桁から255桁に変更する。
48ac6b9 
EC-CUBE#13
@sai-gillingham
e2e テスト。
4444322
@sai-gillingham
e2e テスト。
dcab825
@sai-gillingham
e2e テスト。
b9a3079
@sai-gillingham
一旦、手動のみで実行する。
f890bc4
@sai-gillingham
一旦、手動のみで実行する。
56f1d37
sai-gillingham and others added 27 commits March 15, 2023 12:40
@sai-gillingham
PluginManagerから除外ルーティング設定処理を消す
60683c0
@sai-gillingham
本人認証と二要素認証クラスの別け
e3e9277
@sai-gillingham
\Twig_Environmentを\Twig\Environmentに変更。
9f3b653
@sai-gillingham
性能試験：実行順番変更
2cb0154
@sai-gillingham
性能試験：実行順番変更
5162929
@sai-gillingham
TokenStorageInterfaceは不要のため、消す
2d26c37
@sai-gillingham
フォーム項目の値の抽出をisValid|isSubmittedの範囲内に移動する。
85a1bf5
@sai-gillingham
性能試験：フアイル権限の変更
431b121
@sai-gillingham
findBy -> findOneByに移動
e0a455f
@sai-gillingham
不要変数を消す
2595de3
@sai-gillingham
RedirectResponseを$this->redirectToRouteに変更する。
7b4bd73
@sai-gillingham
性能試験：sudoで過去のフアイルを消す
fa2c288
@sai-gillingham
composer json の更新
3ded762
@sai-gillingham
リポジトリクエリーをfindOneBY関数に変更
3751f20
@sai-gillingham
array_sliceの不具合
2e4d417
@sai-gillingham
plg_two_factor_auth_type.name列をNULLからNOT NULLに変更する。
345ac56
@sai-gillingham
他の会員でも同じ番号によって認証済みになってないかの確認コードの追加。
6018c03
@sai-gillingham
不要変数を消す
6a7f1a3
@sai-gillingham
字の誤りの修正
e077282
@sai-gillingham
不要定数を消す。
c9e6e8d
@kentanakayrglm
本人認証：トークン入力時の電話番号確認処理 考慮漏れを修正
eaf349f
@kentanakayrglm
表記ゆれ修正
d81e346
@kentanakayrglm
概要資料更新
6af3899
@kentanakayrglm
本人認証エンドポイントのチェック修正（認証済みか？→ログイン済みか？）
5931bb6
@kentanakayrglm
本人認証完了時、認証トークンをクリアする
a6f2b8b
@kentanakayrglm
不要コメント削除
ffcbbd4
@kentanakayrglm
フィールド属性を修正
64b80b9
@chihiro-adachi chihiro-adachi merged commit a75a346 into EC-CUBE:main Mar 15, 2023
@sai-gillingham sai-gillingham mentioned this pull request Mar 16, 2023
Closed
@chihiro-adachi chihiro-adachi added this to the 1.0.0 milestone Mar 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.0.0
Development

Successfully merging this pull request may close these issues.
4 participants
@sai-gillingham @chihiro-adachi @kentanakayrglm @KenTanaka