订阅者中间件路由允许公众查看所有受限页面

如何解决订阅者中间件路由允许公众查看所有受限页面

我的项目中的路由和中间件有问题。我有一个仅限订阅者锁定的主题列表。但是，主题可以标记为公开，允许访客用户/公众查看主题。

我面临的问题是我要么让公众看到所有主题，要么不公开。

我有以下订阅者中间件。

public function handle(Request $request,Closure $next)
{
    if ( $request->user() && ! $request->user()->subscribed('annual_membership') ) {
        return redirect('profile');
    } 
        
    if( Auth::Guest() ) {
        return redirect('topics')->with('error','You need to be a registered memeber to view this topic.');
    }

    return $next($request);
}

这是这样添加到我的内核中的。

protected $routeMiddleware = [
    // ...
    'subscriber' => \App\Http\Middleware\Subscriber::class,];

然后我设置了以下路线。

Route::get('voting-topics',[TopicController::class,'topics'])->name('topics');
Route::get('voting-topics/{topic}','topic'])->name('topic');

Route::group(['middleware' => ['subscriber']],function () 
{
    Route::get('profile/billing',function (Request $request) {
        return $request->user()->redirectToBillingPortal();
    });

    Route::get('voting-topics/{topic}','topic'])->name('topic');
});

还有我的主题控制器。

class TopicController extends Controller
{
    public function topics(Request $request) 
    {
        if ( $request->user() && $request->user()->subscribed('annual_membership') ) {
            return $this->subscriberTopics();
        } else {
            return $this->publicTopics();
        }
    }
    
    
    private function publicTopics() 
    {
        $topics = Topic::orderBy('date','asc')->where('free_to_view',1)->paginate(50);

        return view('topics.topics',compact('topics'));
    }
    
    private function subscriberTopics() 
    {
        $topics = Topic::orderBy('date','desc')->paginate(50);

        return view('topics.topics',compact('topics'));
    }

    public function topic(Request $request,Topic $topic) 
    {
        $topic = Topic::find($topic)->first();

        return view('topics.topic',compact(['topic',));
    }
}

Topics 目录工作正常。它只显示标记为公开的来宾和所有订阅者的主题。我的代码目前只允许订阅者查看所有主题，访客可以看到公共主题，这不是我想要的。

如果我删除中间件路由，相反的情况发生，如果客人输入主题的 URL，他们可以访问所有主题。

编辑

为了更好地澄清和解释，我有一个主题列表（基本上就像博客文章）。主题仅供我网站的付费订阅者查看。但是，某些主题可以标记为可免费查看，并且所有人都可以看到。

如果我访问 example.com/voting-topics，那么我只能看到免费主题。 （这是正确的并且有效）如果我登录并转到相同的 URL，我会看到所有主题，因为我已登录，并且我是付费订阅者。 （这再次正确且有效）

如果我删除了中间件中的重复路由，则直接通过 URL 转到 example.com/voting-topics/123 或任何其他主题 ID。我可以查看该主题，即使它应该只对订阅者锁定。

如果我将重复的路由添加回我的中间件，一般公众将无法访问免费主题。

我需要访客/一般公众/非订阅者只能访问标记为公开的主题，但付费订阅者可以访问所有主题。 `

解决方法

所以看起来这毕竟是一个简单的修复，而不是在我的路由中进行检查，我需要在我的控制器中进行检查。

我从中间件中删除了 Route::get('voting-topics/{topic}',[TopicController::class,'topic'])->name('topic');，然后在控制器中添加了以下代码。

if( $topic->free_to_view === 1 || Auth::check() && auth()->user()->subscribed('annual_membership')) {
    return view('topics.topic',compact('topic'));

} else {

    return redirect()->back()->with('error','You need to be a paying member to view this topic');

}