Laravel 重定向不起作用

Question

我正在使用 Laravel 5.1。我的控制器专门用于管理员用户。所以我检查用户是否是管理员。这是我的代码。

public function getAdminData()
  {
    $this->checkAdminStatus();
    return response()->json(array('admin-data'));
  }
public function checkAdminStatus()
  {
    $userManager = new UserManager();
    if(!$userManager->isAdmin())
    {
        return redirect()->route('returnForbiddenAccess');
    }
  }

我的路线是

Route::any('api/app/forbidden',['uses' =>'ErrorController@returnNonAdminErrorStatus','as'=>'returnForbiddenAccess']);

现在如果用户不是管理员，那么它不应该返回 admin-data 但它会返回。 redirect()->route 调用后不应该停止处理逻辑吗？ 这也是纯粹的 REST 应用程序。

Answer 1

您为什么不使用 Laravel 中间件解决方案来满足您的需求？您可以将中间件链接到您的控制器，检查当前用户是否是管理员，如果不是则重定向：

//You Middleware Handle method
public function handle($request, Closure $next)
{
  if ($this->auth->guest() || !($this->auth->user()->isAdmin))
  {
        return redirect('your/url')->with('error','no admin');;
  }
return $next($request);
}

您可以在其构造方法中为控制器添加一个或多个中间件

//your controller
public function __construct(Guard $auth, Request $request){
    $this->middleware('auth', ['except' => ['index', 'show']]); //here one 'auth' middleware
    $this->middleware('admin', ['only' => ['index', 'show', 'create','store']]); //here the admin middleware
}

注意 only 和 except 参数允许或禁止某些控制器方法的中间件

查看中间件上的laravel documentation 了解更多信息：)

Answer 2

不，你的逻辑有点缺陷。您从checkAdminStatus() 发回的返回值只是被忽略并丢弃：

public function getAdminData()
{
    // You don't have $redirectValue in your code, but imagine it 
    // is there. To actually redirect, you need to return this value
    // from your controller method
    $redirectValue = $this->checkAdminStatus();

没有对该重定向进行任何操作。 只有次从你的控制器返回的东西发生在每一个请求上。我会建议更像这样的东西：

public function getAdminData(UserManager $userManager)
{
    if($userManager->isAdmin()) {
        return response()->json(array('admin-data'));
    }

    return redirect()->route('forbidden-access');
}

我认为这抓住了您的问题的精神：这里返回的唯一时间是 如果用户是管理员。

---

顺便说一句，您在一种情况下返回 JSON 数据，在另一种情况下返回 redirect。这可能不是一个好主意。我的理由是，通常情况下，JSON 数据是为了响应 AJAX 请求而返回的，根据我的经验，在失败的情况下很少会跟进 actual 重定向。 ^_(YMMV)