Skip to content

Conversation

@katokaisya
Copy link
Collaborator

@ryuring
後方互換のため、SSL通信時デフォルトは絶対パスを返すようにしています。
ご確認・マージお願いできますでしょうか?

@gondoh gondoh added this to the 4.6.0 milestone Apr 21, 2022
@ryuring ryuring self-requested a review April 23, 2022 05:20
@ryuring ryuring added the Review レビュー要 label Apr 23, 2022
@ryuring
Copy link
Collaborator

ryuring commented Apr 23, 2022

@katokaisya こちらの件、仕様も含めて調べました。

  • Configure の BcEnv.sslUrl というのが、もともと、http と https 共存時代の設定だった
  • https のページから http の URLを出力するために利用されいた(options の ssl を false に設定)
  • 管理画面のSSL化で別途 BcEnv.sslUrl を使うようになった(このタイミングでフルパスがデフォルトになった模様)
  • その後、SSLが標準化され BcEnv.sslUrl は使わなくなった(管理画面で http にアクセスした場合 BcEnv.siteUrlにリダイレクト)

というわけで、こちらの環境では、sslUrl を設定しなければ問題が解決したのですがいかがでしょう?

@ryuring
Copy link
Collaborator

ryuring commented Apr 23, 2022

  • サイト基本設定からは、SSLのURL設定は除外してよさそう
  • 管理画面SSL設定は、セキュリティコンポーネントでSSLを強制する仕様となっているので残しておく

@gondoh gondoh modified the milestones: 4.6.0, Adjusting May 19, 2022
@ryuring ryuring added the Reviewed レビュー済 label Aug 4, 2022
@ryuring ryuring added the Ver4 バージョン4 label Apr 14, 2023
@ryuring ryuring assigned katokaisya and unassigned ryuring Oct 9, 2023
@ryuring
Copy link
Collaborator

ryuring commented Oct 9, 2023

@katokaisya こちらの件、だいぶ前のプルリクで恐縮ですが、レビューを入れてますので確認お願いします。

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

Labels

Review レビュー要 Reviewed レビュー済 Ver4 バージョン4

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants