Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix #1817 [バグ] SSL通信時にBcBaser->getLink()でルートパスで指定していたURLが勝手に絶対パスに書き変え… #1818

Open
wants to merge 2 commits into
base: dev-4
Choose a base branch
from

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