Skip to content

docs: Docker 環境の HTTPS アクセス手順を README に追記#6944

Open
ttokoro20240902 wants to merge 2 commits into
4.4from
docs/readme-https-local-access
Open

docs: Docker 環境の HTTPS アクセス手順を README に追記#6944
ttokoro20240902 wants to merge 2 commits into
4.4from
docs/readme-https-local-access

Conversation

@ttokoro20240902

@ttokoro20240902 ttokoro20240902 commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

概要(Overview・Refs Issue)

docker compose ... up -d で起動した開発環境のアクセス方法について、README が http://localhost:8080 のみを案内していますが、この URL ではログインできません

dev/prod 環境のセッションクッキーは app/config/eccube/packages/framework.yamlcookie_samesite: none(+ cookie_secure: auto)で発行されます。SameSite=None のクッキーはブラウザ仕様上 Secure 属性(=HTTPS 接続)が必須で、HTTP 接続だとクッキーが破棄されるため、ログイン・カートなどセッションを使う操作が成立しません。

実際に必要な HTTPS(https://127.0.0.1:4430/ = docker-compose.yml の 4430:443)でのアクセス方法と、自己署名証明書(Apache snakeoil)による「保護されていない通信」警告の解消手順を追記します。

方針(Policy)

  • 日本語インストール節に「Docker 環境へのアクセス」小節を新設し、アクセス URL・HTTP でログイン不可の理由・mkcert による警告解消手順を記載。
  • 英語版 Quick Start の # Access http://localhost:8080 を HTTPS 案内へ訂正。
  • ドキュメント(README.md)のみの変更で、コード・設定・挙動の変更はありません。

実装に関する補足(Appendix)

  • HTTPS 化(a2enmod ssl / default-ssl.conf / EXPOSE 443)は Dockerfile に以前から存在します。今回はコードは変えず、README の案内を実態に合わせるものです。
  • mkcert 手順は任意(警告を消したい開発者向け)で、証明書はコミットしない旨を注記しています。

テスト(Test)

ドキュメントのみの変更のため、テスト・静的解析の対象はありません。

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

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

🤖 Generated with Claude Code

Summary by CodeRabbit

  • ドキュメント
    • Docker環境でのHTTPSアクセス手順(アクセス先の切替を含む)を追加しました。
    • HTTPではログインできない理由(SameSite cookieによりHTTPS必須)を明記しました。
    • 自己署名証明書による警告の扱いと、mkcertでローカル信頼証明書を発行してコンテナへ反映する手順を追記しました。

`docker compose up -d` 起動後のアクセス URL が README で
`http://localhost:8080` のみ案内されていたが、dev/prod のセッションクッキーは
`cookie_samesite: none`(+ `cookie_secure: auto`)で発行されるため、
HTTP 接続ではブラウザがクッキーを破棄しログインできない。

実際に必要な HTTPS(`https://127.0.0.1:4430/`)でのアクセスと、
自己署名証明書による「保護されていない通信」警告を mkcert で解消する手順を
日本語インストール節に追記し、英語版 Quick Start の誤った案内も訂正する。

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jul 17, 2026

Copy link
Copy Markdown

Review Change Stack

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: 3916cdf5-e08e-4f54-8dcb-513716e17545

📥 Commits

Reviewing files that changed from the base of the PR and between 14b1dcf and 5b0b884.

📒 Files selected for processing (1)
  • README.md
🚧 Files skipped from review as they are similar to previous changes (1)
  • README.md

📝 Walkthrough

Walkthrough

README.md に Docker 環境の HTTPS アクセス先、HTTP ログイン制約、自己署名証明書の扱い、mkcert による証明書差し替え手順を日本語・英語で追記しました。

Changes

Docker HTTPS ドキュメント

Layer / File(s) Summary
Docker HTTPS アクセス手順
README.md
HTTPS のアクセス先、SameSite=None により HTTP ではログインできない理由、自己署名証明書の扱い、mkcert による証明書差し替え手順を日本語・英語で追記しました。

Estimated code review effort: 1 (Trivial) | ~5 minutes

Poem

ぴょんと HTTPS、扉を開け
クッキー守って、月明かり
証明書も mkcert
鍵はそっと秘密のまま
README 片手に跳ねてゆく

🚥 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 README に Docker 環境の HTTPS アクセス手順と関連説明を追記する変更を的確に表しています。
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/readme-https-local-access

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.

WSL 内で mkcert -install しても Windows 側ブラウザ(Chrome/Edge は
Windows の証明書ストアを参照)には反映されないため、rootCA を Windows へ
取り込む手順を mkcert 節に補足する。

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nanasess

Copy link
Copy Markdown
Contributor

詳細は doc4 に記載すべき内容かなと思います。 docker-compose.yml の - 8080:80 は削除してしまった方がすっきりするかもしれません
https://doc4.ec-cube.net/quickstart/install

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.

2 participants