The GET method is not supported for this route. Supported methods: POST?

Question

[ildar-meyker]

Есть у меня такой route:

Route::post('authors/{author}/delete-poster', 'AuthorController@deletePoster');

Я его добавил, чтобы удалять постер автора (картинка).

Потом решил зайти через get запрос, чтобы посмотреть, а что же будет. И мне сгенерировалось сообщение из заголовка. Так-то все верно, такого роута действительно нет. Но это же не нормально, да? Мне следовало использовать другой метод или что?

Comments

[kafkiansky]

Что значит ненормально-то? Удаление идёт по посту, вы заходите гетом. Что вы ожидали, веселье и хлопушки?

[ildar-meyker]

kafkiansky, Я ожидал что-то типа 404 и меня бы это устроило.

[kafkiansky]

ildar-meyker, роут есть, обращение к нему неверное. Откуда там 404 будет? Веб сервер правильно сгенерировал ошибку.

[ildar-meyker]

kafkiansky, но я же определяю роут для post-запроса. C фига ли считается, что я хочу что-то получить через get? По факту я ведь и в режиме продакшена получу ошибку. Поэтому тут либо я неверно все организовал, либо хз.

[Daria Motorina]

ildar-meyker, статус ошибки будет 405, а не 404 - все верно, на проде будет отдаваться стандартный текст для юзера емнип. Если юзер сам лезет в урлы и пытается гетом дергать эндпоинт, который предназначен для метода пост - то это его проблемы

[kafkiansky]

Daria Motorina, поддерживаю. На проде в день по 10 ошибок с неправильным запросом роутов. Это нормально.

Answer 1

[Иван Шумов]

Все логично. Фреймворк работает по таким правилам, а не как вам захотелось. Хочется сделать поведение, которое ожидаете - настраиваем роутинг дальше.

Comments

[ildar-meyker]

Допустим так. Я собственно не против. Как мне следовало организовать роутинг? Принимать get запрос и выбрасывать ошибку?

[Иван Шумов]

ildar-meyker, проблему уже решали 100500 раз

[ildar-meyker]

Сам подход к удалению изображения вообще верный?

[Иван Шумов]

ildar-meyker, если по популярным практикам то, конечно, нет.
DELETE /posters/:poster_id
выглядело бы куда логичнее

[ildar-meyker]

Иван Шумов, у меня нет модели для постеров. Поэтому позволительно, наверное, да?

[Иван Шумов]

ildar-meyker, твой код-твои правила, но вообще-то за такое сегодня бьют по рукам. За необоснованную лень

[Daria Motorina]

ildar-meyker, нет, эндпоинт вида /entity/entity-id не обязан строго соответствовать существованию модели entity.
Так, как у вас сейчас, можно переделать роутинг с
POST authors/{author}/delete-poster
на
DELETE authors/{author}/posters/{poster-id}
перед этим допилив урл GET authors/{author}/posters, чтобы видеть список постеров у автора

[Иван Шумов]

Daria Motorina, ну, выглядит хорошо, но внутри образовывается противненький такой монолит. Потом множество болей будет у человека же, ну.

[Daria Motorina]

Иван Шумов, у нас так делали, когда управляли связанной сущностью (как постеры у автора). Думаете стОит управлять связанной сущностью отдельными эндпоинтами на уровне апи? Я просто не встречала таких примеров в реальных публичных апи

[Иван Шумов]

Daria Motorina, да. По тому что роутинг это абстрактная величина. Мы можем написать там что угодно, но чем больше мы в него зависимостей вставляем тем больше нам надо знать про объект. Например в том что я вижу нам надо знать и владельца постера и сам постер. Вопрос - зачем? Отвечаю сразу - никому это не сдалось.

[Daria Motorina]

Иван Шумов, спасибо. есть над чем подумать)

Answer 2

[Вячеслав Плиско]

Всё верно метод GET только для получения данных, а POST(PUT, DELETE и т.д.) для изменения данных. Это важно с точки зрения безопасности, к тому же POST надо подпирать CSRF токеном.

Comments

[Аль Капоне]

благодарю, недоглядел) в web.php post на get поменял и заработало