diff --git a/resources/views/admin/site_subscriptions/index.blade.php b/resources/views/admin/site_subscriptions/index.blade.php
index 97c358b..fd71f0c 100644
--- a/resources/views/admin/site_subscriptions/index.blade.php
+++ b/resources/views/admin/site_subscriptions/index.blade.php
@@ -61,19 +61,31 @@
用于运营快速定位需要处理的订阅集合(口径基于筛选条件组合)。
@php
- // 快捷筛选:仅保留“上下文”字段(站点/套餐/back/关键词),避免把其它筛选条件叠加导致空结果
- $buildQuickFilterUrl = function (array $overrides) {
+ // back 安全护栏:本页大量链接使用 `{!! !!}` 原样输出,必须严控 back 注入与 nested back。
+ $incomingBack = (string) request()->query('back', '');
+ $safeBackForLinks = (str_starts_with($incomingBack, '/')
+ && !preg_match('/["\'<>]/', $incomingBack)
+ // back 本身不应再包含 back(避免无限嵌套导致 URL 膨胀)
+ && !preg_match('/(?:^|[?&])back=/', $incomingBack))
+ ? $incomingBack
+ : '';
+
+ // 快捷筛选:仅保留“上下文”字段(站点/套餐/keyword/安全 back),避免把其它筛选条件叠加导致空结果
+ $buildQuickFilterUrl = function (array $overrides) use ($safeBackForLinks) {
$path = '/' . ltrim(request()->path(), '/');
$contextKeys = [
'merchant_id' => 1,
'plan_id' => 1,
- 'back' => 1,
'keyword' => 1,
];
$q = array_intersect_key(request()->query(), $contextKeys);
+ if ($safeBackForLinks !== '') {
+ $q['back'] = $safeBackForLinks;
+ }
+
foreach ($overrides as $k => $v) {
if ($v === null) {
unset($q[$k]);
@@ -89,17 +101,10 @@
return $path . '?' . \Illuminate\Support\Arr::query($q);
};
- // “全部”:清空筛选,但保留 back(用于返回来源页)
- $incomingBack = (string) request()->query('back', '');
- $safeBack = (str_starts_with($incomingBack, '/')
- && !preg_match('/["\'<>]/', $incomingBack)
- // back 本身不应再包含 back(避免无限嵌套导致 URL 膨胀)
- && !preg_match('/(?:^|[?&])back=/', $incomingBack))
- ? $incomingBack
- : '';
+ // “全部”:清空筛选,但保留安全 back(用于返回来源页)
$allUrl = '/admin/site-subscriptions';
- if ($safeBack !== '') {
- $allUrl .= '?' . \Illuminate\Support\Arr::query(['back' => $safeBack]);
+ if ($safeBackForLinks !== '') {
+ $allUrl .= '?' . \Illuminate\Support\Arr::query(['back' => $safeBackForLinks]);
}
@endphp
diff --git a/tests/Feature/AdminSiteSubscriptionIndexUnsafeBackShouldBeDroppedForLinksTest.php b/tests/Feature/AdminSiteSubscriptionIndexUnsafeBackShouldBeDroppedForLinksTest.php
new file mode 100644
index 0000000..a608568
--- /dev/null
+++ b/tests/Feature/AdminSiteSubscriptionIndexUnsafeBackShouldBeDroppedForLinksTest.php
@@ -0,0 +1,47 @@
+seed();
+
+ $this->post('/admin/login', [
+ 'email' => 'platform.admin@demo.local',
+ 'password' => 'Platform@123456',
+ ])->assertRedirect('/admin');
+ }
+
+ public static function invalidBackProvider(): array
+ {
+ return [
+ 'contains quote' => ['/' . 'admin/site-subscriptions?x="y"'],
+ 'contains angle bracket' => ['/admin/site-subscriptions?x=