メンテナンス画面のHTTPヘッダーのバージョン表記を実際のプロトコルに合わせる#6629
Conversation
HTTPバージョンがHTTP/1.1にハードコードされていたため、 http_response_code()を使用して実際のプロトコルバージョンを反映するように修正 fixes EC-CUBE#6407 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 📝 WalkthroughWalkthroughメンテナンスモード時のHTTPステータスコード設定方法を変更。 Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 6✅ Passed checks (6 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
Tip Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord. 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. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## 4.3 #6629 +/- ##
============================================
- Coverage 78.84% 78.83% -0.02%
Complexity 6631 6631
============================================
Files 475 475
Lines 26539 26539
============================================
- Hits 20924 20921 -3
- Misses 5615 5618 +3
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
概要
メンテナンス画面表示時のHTTPレスポンスヘッダーで、HTTPバージョンが
HTTP/1.1にハードコードされていたため、実際のプロトコルバージョン(HTTP/2, HTTP/3等)を反映するように修正しました。再現手順
.maintenanceファイル作成)HTTP/1.1 503となっている原因
index.phpの75行目でheader('HTTP/1.1 503 Service Temporarily Unavailable');とHTTPバージョンがハードコードされていた。修正内容
http_response_code(503)を使用することで、PHPが自動的に実際のHTTPプロトコルバージョンを使用するように変更。検証方法
影響範囲
index.phpのメンテナンス画面表示処理のみ互換性
破壊的変更なし。動作に影響はなく、レスポンスヘッダーのプロトコル表記が正確になるのみ。
fixes #6407
🤖 Generated with Claude Code
Summary by CodeRabbit
リリースノート