diff --git a/resources/views/admin/site_subscriptions/show.blade.php b/resources/views/admin/site_subscriptions/show.blade.php index 03774a4..66e4e5d 100644 --- a/resources/views/admin/site_subscriptions/show.blade.php +++ b/resources/views/admin/site_subscriptions/show.blade.php @@ -121,7 +121,12 @@ @php $back = (string) request()->query('back', ''); // back 安全校验:只接受相对路径,且拒绝引号/尖括号(由于下方 href 采用原样输出以避免 & 影响回链/断言) - $safeBack = (str_starts_with($back, '/') && !preg_match('/["\'<>]/', $back)) ? $back : ''; + $safeBack = (str_starts_with($back, '/') + && !preg_match('/["\'<>]/', $back) + // back 本身不应再包含 back(避免无限嵌套导致 URL 膨胀) + && !preg_match('/(?:^|[?&])back=/', $back)) + ? $back + : ''; @endphp @if($safeBack) diff --git a/tests/Feature/AdminSiteSubscriptionShowBackValidationTest.php b/tests/Feature/AdminSiteSubscriptionShowBackValidationTest.php new file mode 100644 index 0000000..188f5b7 --- /dev/null +++ b/tests/Feature/AdminSiteSubscriptionShowBackValidationTest.php @@ -0,0 +1,63 @@ +seed(); + + $this->post('/admin/login', [ + 'email' => 'platform.admin@demo.local', + 'password' => 'Platform@123456', + ])->assertRedirect('/admin'); + } + + public function test_show_should_not_render_back_link_when_back_contains_nested_back_param(): void + { + $this->loginAsPlatformAdmin(); + + $merchant = Merchant::query()->firstOrFail(); + $plan = Plan::query()->create([ + 'code' => 'sub_show_back_nested_plan', + 'name' => '订阅详情 nested back 校验测试套餐', + 'billing_cycle' => 'monthly', + 'price' => 10, + 'list_price' => 10, + 'status' => 'active', + 'sort' => 10, + 'published_at' => now(), + ]); + + $sub = SiteSubscription::query()->create([ + 'merchant_id' => $merchant->id, + 'plan_id' => $plan->id, + 'status' => 'active', + 'source' => 'manual', + 'subscription_no' => 'SUB_SHOW_BACK_NESTED_0001', + 'plan_name' => $plan->name, + 'billing_cycle' => $plan->billing_cycle, + 'period_months' => 1, + 'amount' => 10, + 'starts_at' => now(), + 'ends_at' => now()->addMonth(), + ]); + + $nestedBack = '/admin/site-subscriptions?status=active&back=/admin/platform-orders'; + + $res = $this->get('/admin/site-subscriptions/' . $sub->id . '?back=' . urlencode($nestedBack)); + $res->assertOk(); + + $res->assertDontSee('返回上一页(保留上下文)'); + $res->assertDontSee($nestedBack, false); + } +}