-
Notifications
You must be signed in to change notification settings - Fork 645
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.
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
管理画面へ2段階認証機能を追加 #5017
管理画面へ2段階認証機能を追加 #5017
Conversation
Codecov Report
@@ Coverage Diff @@
## 4.1-feature #5017 +/- ##
==================================================
- Coverage 76.38% 39.82% -36.57%
- Complexity 6190 6255 +65
==================================================
Files 445 449 +4
Lines 20963 21137 +174
==================================================
- Hits 16013 8417 -7596
- Misses 4950 12720 +7770
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## 4.1 #5017 +/- ##
============================================
- Coverage 75.96% 75.50% -0.46%
- Complexity 5946 6013 +67
============================================
Files 449 453 +4
Lines 20922 21098 +176
============================================
+ Hits 15893 15930 +37
- Misses 5029 5168 +139
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
@sw-satoshi-nakano ありがとうございます!取り込ませていただきました。 |
@okazy @chihiro-adachi |
概要(Overview・Refs Issue)
#4850
方針(Policy)
EC-CUBEの他の機能と同様なファイル構造で開発した。
実装に関する補足(Appendix)
追加した画面
ログイン後(またはログイン中)に2段階認証をしていない時にこの画面を表示します。
2段階認証を設定しない状態でログインした場合(またはログイン中)は、この画面を表示します。
(右上の「2段階認証 設定」ボタンは有効にした場合のみ表示するようにしています)
その他
テスト(Test)
相談(Discussion)
2段階認証済みであるかの確認をクッキーを使って行っていますが、クッキーに保存する値は、ログインパスワードを生成する時のロジックを使って暗号化しています。気を付けた方が良いことあれば、ご指摘いただきたいです。
コード:Eccube\Service\TwoFactorAuthService::createAuthedCookie
マイナーバージョン互換性保持のための制限事項チェックリスト
レビュワー確認項目