destroy 方法里的 授权策略 是不是多余的?

我们在视图里使用了 Blade 命令 @can('delete', $user)

UserPolicy 策略里的 destroy() 方法已经做出了判断:

public function destroy(User $currentUser, User $user)
{
    return $currentUser->is_admin && $currentUser->id !== $user->id;
}

这里的 $currentUser->is_admin 不就已经可以让普通用户不显示删除按钮了吗?

想问的问题是:

  1. UsersController 里的 destroy() 方法里的 $this->authorize() 是不是不加也可以?
  2. 如果加,有什么充分的理由吗?
  3. 如果不加,会有什么影响吗?
  4. 最佳实践是哪种呢?

其实我觉得加了更安全,不过一方面是用更少的代码达到相同的效果,一方面是看起来更安全的可能是画蛇添足,取舍不下。

小时候,幸福是一件很简单的事;长大了,简单是一件很幸福的事。
《L01 基础入门》
我们将带你从零开发一个项目并部署到线上,本课程教授 Web 开发中专业、实用的技能,如 Git 工作流、Laravel Mix 前端工作流等。
《G01 Go 实战入门》
从零开始带你一步步开发一个 Go 博客项目,让你在最短的时间内学会使用 Go 进行编码。项目结构很大程度上参考了 Laravel。
你看我吊吗啊
最佳答案

@Soldoros 你还较真了,很多工具都可以模拟post、delete请求的 自己百度吧。

5年前 评论
讨论数量: 8
你看我吊吗啊

@Soldoros 你还较真了,很多工具都可以模拟post、delete请求的 自己百度吧。

5年前 评论
你看我吊吗啊

你这段代码只是不显示删除按钮,但是用户可以尝试直接在浏览器地址栏 去访问删除操作的函数的,策略里是防止这个。

5年前 评论
月光

@JeffLi 提交方式是 DELETE ,需要 POST 方式提交吧,地址栏怎么能提交?

5年前 评论
你看我吊吗啊

@Soldoros 你还较真了,很多工具都可以模拟post、delete请求的 自己百度吧。

5年前 评论
你看我吊吗啊

@Soldoros

file 你是用手机回复的吗。。

5年前 评论
月光

@JeffLi 嗯呢,没想到这个样子,一个大拇指

5年前 评论
你看我吊吗啊

采纳我的答案,不然还有人进来看问题

5年前 评论

讨论应以学习和精进为目的。请勿发布不友善或者负能量的内容,与人为善,比聪明更重要!