Как сделать маршрутизацию в Laravel более строгой?

Question

[Сергей delphinpro]

Есть такой роут

Route::get('a/{article}', [ArticleController::class, 'show'])
    ->name('article.show');

Я ожидаю, что под него будут попадать урлы типа /a/7, /a/123 и т.д.

На деле, например, страница /a/7 также открывается по любому из адресов, содержащему буквенные символы после цифры: /a/7asd, /a/7qwerty и т.д. Открываются с кодом 200.

Как это исправить? И вообще, какого хрена так происходит?

Answer 1

[smilingcheater]

Читайте доки, в доках всё описано. https://laravel.com/docs/11.x/routing#parameters-r...

Comments

[Сергей delphinpro]

Да, про where я знаю. Получается, нужно будет в каждом роуте ограничения прописывать? У меня их пару десятков. Глобальные тоже вряд ли подойдут, там же разные имена параметров, получится те же два десятка ограничений, только в провайдере.

[smilingcheater]

Сергей delphinpro, Читаем доки дальше :)
https://laravel.com/docs/11.x/routing#parameters-g...
Можете объявить глобальное правило для параметра с определённым названием, и указывать именно его в роутах

[smilingcheater]

Сергей delphinpro, т.е. у вас везде разные имена параметров, и на какие-то надо ограничение, а на какие-то нет? И как по вашему тут в принципе возможно понять, где надо ограничивать, а где нет?

[Сергей delphinpro]

smilingcheater,
ну к примеру три маршрута (не будем десятки перечислять, но представим, что их много)

Route::get('a/{article}'
Route::get('b/{product}'
Route::get('c/{supplier}'

по доке я вижу два варианта

Route::get('a/{article}')->whereNumber('article')
Route::get('b/{product}')->whereNumber('product')
Route::get('c/{supplier}')->whereNumber('supplier')

или

public function boot(): void
{
    Route::pattern('article', '[0-9]+');
    Route::pattern('product', '[0-9]+');
    Route::pattern('supplier', '[0-9]+');
}

Но по сути они одинаковы. И там и там придется прописывать все ограничения.
Хотелось бы как-то один раз описать.

[smilingcheater]

Сергей delphinpro, Давайте зайдём с другой стороны. Основная проблема - это 200 статус? Покажите код экшна, как из параметра роута вы получаете инстанс модели. DI или руками?

[Сергей delphinpro]

smilingcheater, дефолтный автобиндинг

public function show(Article $article) {
}

основная проблема - дубликаты страниц. она решается прописыванием ограничений в каждом роуте.

вторичная проблема, которую и пытаюсь теперь решить - код. ну не хочется мне в каждом роуте писать ограничения =)

как вариант, можно использовать одно и то же имя параметра, {id} и прописать единственное глобальное ограничение в провайдере. но тогда и в контроллерах придется писать нечто вроде

function show(Article $id) {
  echo $id->id; // Fuck =((
}

некрасиво, не нравится.

Или от неявного биндинга отказываться.

function show($id) {
  if (!($article = Article::find($id)) {
    abort(404)
  }
}

но тут модель придется искать в каждом подобном методе. Писанины не меньше, что сводит на нет всю идею.

[smilingcheater]

Сергей delphinpro, Можно попробовать в моделях переопределить метод resolveRouteBinding, дописать строгость матчинга по ID
https://laravel.com/docs/11.x/routing#customizing-...
Можно это переопределение оформить трейтом, и подключать к нужным моделям

[Антон Антон]

Мне кажется, единственное, зачем это можно сделать - как раз убрать запросы к БД при кривых параметрах.
А откуда там дубли и всё остальное - хз.

В этом плане вариант с примесью также позволит убрать запросы к БД, как и ограничения маршрута.

[tukreb]

В симфони так же для каждого роутера прописываются ограничения, откуда они могут знать что в ID вы ждёте, цифры или буквы.

#[Route(path: '/{id}/edit', name: '.edit', requirements: ['id' => Requirement::ULID])]