remember_meにsecure属性を設定

[chihiro-adachi]

概要(Overview・Refs Issue)

remember_meにsecure属性を設定するように対応

方針(Policy)

実装に関する補足(Appendix)

テスト（Test)

secure属性が有効になっていることを確認

相談（Discussion）

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

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

レビュワー確認項目

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

[codecov-commenter]

Codecov Report

Merging #5769 (4b49277) into 4.2 (01ff147) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                4.2    #5769   +/-   ##
=========================================
  Coverage     78.86%   78.86%           
  Complexity     6275     6275           
=========================================
  Files           469      469           
  Lines         21078    21078           
=========================================
  Hits          16623    16623           
  Misses         4455     4455           

Flag	Coverage Δ
E2E	64.86% <ø> (ø)
Unit	77.56% <ø> (-0.05%)	⬇️

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

Impacted Files	Coverage Δ
...be/Service/PurchaseFlow/Processor/TaxProcessor.php	82.45% <0.00%> (-1.76%)	⬇️
src/Eccube/Entity/TaxRule.php	86.41% <0.00%> (+1.23%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[kiy0taka]

@chihiro-adachi この修正だとhttp通信のときはRemember-Me使えなくなると思いますが、問題ないでしょうか？

[chihiro-adachi]

@kiy0taka
本番環境でhttp運用はあまり考えられないので、httpで動作しないのはまあ許容範囲かなと。

一応以下の箇所でCookieを発行していて、http/httpsで有効無効を切り替えてくれるように見えるんですが、$this->options['secure']が未定義になることはなく、またsecurity.yamlでnullを設定することもできずで、ここは通らないコードのようでした。
https://github.com/symfony/symfony/blob/5.4/src/Symfony/Component/Security/Http/RememberMe/AbstractRememberMeHandler.php#L123

[kiy0taka]

@chihiro-adachi auto でいけそう。
https://github.com/symfony/symfony/blob/5.4/src/Symfony/Bundle/SecurityBundle/DependencyInjection/Security/Factory/RememberMeFactory.php#L112

[chihiro-adachi]

@kiy0taka
なんと。ありがとうございます！確認してみます。

[chihiro-adachi]

@kiy0taka
autoで有効無効切り替えできるのを確認しました。
ありがとうございました！

[kiy0taka]

ありがとうございました。