Skip to content

chore: bump TwoFactorAuth to v2#6701

Open
dotani1111 wants to merge 2 commits intoEC-CUBE:4.3from
dotani1111:chore/bump-twofactorauth-2
Open

chore: bump TwoFactorAuth to v2#6701
dotani1111 wants to merge 2 commits intoEC-CUBE:4.3from
dotani1111:chore/bump-twofactorauth-2

Conversation

@dotani1111
Copy link
Copy Markdown
Contributor

@dotani1111 dotani1111 commented Apr 1, 2026

DependabotのPRがうまくいってなかったので、こちらで上げ直しました。
#6163

概要(Overview・Refs Issue)

方針(Policy)

実装に関する補足(Appendix)

テスト(Test)

相談(Discussion)

マイナーバージョン互換性保持のための制限事項チェックリスト

  • 既存機能の仕様変更はありません
  • フックポイントの呼び出しタイミングの変更はありません
  • フックポイントのパラメータの削除・データ型の変更はありません
  • twigファイルに渡しているパラメータの削除・データ型の変更はありません
  • Serviceクラスの公開関数の、引数の削除・データ型の変更はありません
  • 入出力ファイル(CSVなど)のフォーマット変更はありません

レビュワー確認項目

  • 動作確認
  • コードレビュー
  • E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
  • 互換性が保持されているか
  • セキュリティ上の問題がないか
    • 権限を超えた操作が可能にならないか
    • 不要なファイルアップロードがないか
    • 外部へ公開されるファイルや機能の追加ではないか
    • テンプレートでのエスケープ漏れがないか

Summary by CodeRabbit

リリースノート

  • 保守作業
    • サードパーティ製二段階認証ライブラリを更新しました。
    • セキュリティと互換性の改善により、認証フローの安定性と信頼性が向上します。
    • 既存の他ライブラリや設定に変更はなく、ユーザー向けの操作変更はありません。

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 1, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 54bc852b-1cd4-4a4a-8b56-6b16bb626cba

📥 Commits

Reviewing files that changed from the base of the PR and between 37227ad and c881ce2.

⛔ Files ignored due to path filters (1)
  • composer.lock is excluded by !**/*.lock
📒 Files selected for processing (1)
  • composer.json
🚧 Files skipped from review as they are similar to previous changes (1)
  • composer.json

📝 Walkthrough

Walkthrough

composer.jsonにおいて、robthree/twofactorauthの依存関係制約が^1.8から^2.0にアップグレードされました。Composerが解決・インストールするパッケージのバージョンが変更されます。

Changes

Cohort / File(s) Summary
依存関係の更新
composer.json
robthree/twofactorauthの依存関係制約を^1.8から^2.0にアップグレード

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested labels

dependencies

Suggested reviewers

  • saori-kakiuchi
  • nanasess
  • ji-eunsoo

Poem

🐰 ふわり、ふわり〜
バージョン上げて、二要素認証も進化する日
^2.0へ跳ねる、小ウサギのごとく
セキュリティ強化で、みんなニッコリ ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed プルリクエストのタイトルはTwoFactorAuth依存関係をv2にバージョンアップすることが明確に示されており、raw_summaryの変更内容と完全に一致しています。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@dotani1111 dotani1111 added this to the 4.4.0 milestone Apr 1, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 1, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.81%. Comparing base (dc22eef) to head (c881ce2).
⚠️ Report is 30 commits behind head on 4.3.

Additional details and impacted files
@@            Coverage Diff            @@
##                4.3    #6701   +/-   ##
=========================================
  Coverage     78.80%   78.81%           
+ Complexity     6825     6824    -1     
=========================================
  Files           476      476           
  Lines         27711    27709    -2     
=========================================
+ Hits          21838    21839    +1     
+ Misses         5873     5870    -3     
Flag Coverage Δ
Unit 78.81% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@dotani1111
Copy link
Copy Markdown
Contributor Author

二要素認証の周辺をチェックする

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant