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

管理画面をIP制限した際に表示される画面のリンク先が 'localhost' になる #4561

Closed
okazy opened this issue Jun 8, 2020 · 3 comments
Labels
Milestone

Comments

@okazy
Copy link
Contributor

okazy commented Jun 8, 2020

概要(Overview)

設定>システム設定>セキュリティ管理 からIP制限をかけ、許可されていないIPからアクセスすると「アクセスできません。」の画面が表示される。
そこに表示されている「ログイン画面へ」のリンクが 'localhost' になってしまっている。

image

期待する内容(Expect) or 要望 (Requirement)

対応は3ケースほど考えられる。

  • 「ログイン画面へ」のリンクを管理画面のログイン画面のリンクにする
  • 「ログイン画面へ」のリンクをフロント画面トップページのリンクにする
  • 管理画面が外部からわからないようにそもそもフロント画面の404等にする。

再現手順(Procedure)

設定>システム設定>セキュリティ管理 からIP制限をかける。
許可されていないIPから管理画面へアクセスする。

環境 (environment)

  • EC-CUBE: 4.0.3
  • PHP: 7.4
  • DB:
    • MySQL 5.7

関連情報 (Ref)

@okazy okazy added the bug:Low label Jun 8, 2020
@okazy okazy added this to the 4.0.x milestone Jun 8, 2020
@okazy
Copy link
Contributor Author

okazy commented Jun 25, 2020

原因を調べて見ましたがわかりませんでした。
こちらのエラー画面は403以外にも管理画面のHttpException系は全てこのテンプレートを利用してるようです。
404では問題なく動いていますので、403のためにリンク先を切り替えるのは冗長な気がしますね。。。

@dk-umebius
Copy link
Contributor

リンクがlocalhostになるのはIpAddrListenerのリスナーのpriorityが512のためで、31以下でしたら正しいドメインになるのではないかと思います。
https://github.com/EC-CUBE/ec-cube/blob/4.0.4/src/Eccube/EventListener/IpAddrListener.php#L62

ただpriority変えるのはプラグイン等に影響出そうですし、twigのurlをpathに変えることで対応可能かと思いましたので一応プルリクさせていただきました。
どんな修正方法でも良いのですが、アクセス制限画面でリンク切れがあまり印象良くないので何かしら対応いただけるとありがたいです。

@kiy0taka
Copy link
Contributor

#4637 で対応いただいたのでクローズします。

@okazy okazy modified the milestones: 4.0.x, 4.0.5 Aug 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants