baserproject / basercms

baserCMS : Based Website Development Project
http://basercms.net
Other
165 stars 126 forks source link

【API】BcUtil::isAdminSystem でAPI経由で処理しているURLが管理画面ではないと判断されてしまうようです #3012

Open kaburk opened 7 months ago

kaburk commented 7 months ago

概要

イベント処理などでBcUtil::isAdminSystemで判別しようとしたときに発生しました。 内部の処理を追ってみるとコンテンツ管理などでAPIで処理しているURLの判別が必要な時に APIのURLなのでプレフィックスが異なる扱いになってしまい管理画面ではないと判断されてしまうようです。

isAdminSystemの処理の後半の正規表現のチェックにAPI経由の場合の管理画面プレフィックス(/baser/api/admin/)のチェックを追加すると対応できそうです。

スクリーンショット 2024-01-05 20 06 43

baserCMS version : 5.0.8-dev

ryuring commented 7 months ago

@kaburk 報告ありがとうございます。管理画面と管理用APIは分けた方がよいかもと考えていますがいかがでしょう? isAdminApi() を作るとか。

kaburk commented 7 months ago

@ryuring isAdminApi() あると良さそうですね!

ryuring commented 7 months ago

@kaburk 実装お願いしますw