SSL強制でないときは、http,httpsどちらでも受け入れるように修正。 #3836
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
概要(Overview・Refs Issue)
SSL強制を指定していないときは、スキーマがhttpに固定されていましたが、http/httpsのどちらでも受け入れるように変更しました。
これにより、httpsでアクセスしてもhttpにリダイレクトされてしまう挙動が直り、http/httpsのどちらでもアクセスできるようになります。
SLL強制=true のときは、今までどおり https 固定です。
方針(Policy)
実装に関する補足(Appendix)
3系は以下のコードでした。
https://github.com/EC-CUBE/ec-cube/blob/master/src/Eccube/Application.php#L575
テスト(Test)
にて、
httpsアクセスでインストーラ起動して、SSL強制できることを確認
管理画面からSSL強制をON/OFFできることを確認
また、
IS_AUTHENTICATED_FULLY などの権限管理が動作していることを確認しました。
相談(Discussion)
cookieのセキュア属性は、はSSL強制のオプションに連動するままです。
マイナーバージョン互換性保持のための制限事項チェックリスト