z-song / laravel-admin

Build a full-featured administrative interface in ten minutes
https://laravel-admin.org
MIT License
11.15k stars 2.82k forks source link

fix: empty http path and method in permission allows any route open; #5741

Open alexoleynik0 opened 1 year ago

alexoleynik0 commented 1 year ago

Original issue -- #5739 I can't tell why this was designed like that, and the chances that after 6+ years it's safe to change it are not very high. Still, there's really no indication anywhere on the "Permission create" page that any Permission without http_path and http_method filled will give full access to any route and method. It may be better to restrict empty values there (eg make http_path required) to prevent this.