Laravel API: url matching in api.php file

Question

I am creating an API with Laravel 8 and in the requests to the different resources the path mapping does not behave as I would expect.

The entities are cameras and alert levels. A camera can have several alert levels.

The routes I have in the api.php file are:

//get the information of a specific camera. Works fine.
Route::get('cameras/{id}', [CameraController::class, 'getCamera']);

//get (all) alert levels information from all cameras
Route::get('cameras/alert-levels', [AlertLevelController::class, 'getCamerasAlertLevels']);

When I access this route:

http://www.example.com/api/v1/cameras/alert-levels

This route should match the second of the routes but however it is matching the first one.

If I alter the order of the routes they work correctly but I don't think I should have to do that. That is, it is supposed to detect that the URI does not match the first of the paths and that it does match the second.

Am I making a mistake?

Thank you very much in advance.

shaedrich · Accepted Answer · 2021-06-04 11:04:46Z

1

I bet, your id is numeric, so you can do the following using parameter constraints:

//get the information of a specific camera. Works fine.
Route::get('cameras/{id}', [CameraController::class, 'getCamera'])->where('id', '\d+'); // or ->whereNumber();

//get (all) alert levels information from all cameras
Route::get('cameras/alert-levels', [AlertLevelController::class, 'getCamerasAlertLevels'])

answered Jun 4, 2021 at 11:04

shaedrich

5,7633 gold badges31 silver badges47 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

John Lobo · Accepted Answer · 2021-06-04 11:11:20Z

0

You should change order of routes.Because both get request and it matches first one so it goes to cameras/{id}.

so dynamic param string then we must kept last.or else you can follow other answer mentioned by @shaedrich.

for security reason usually id passed as encrypted so it will generate long string

Route::get('cameras/alert-levels', [AlertLevelController::class, 'getCamerasAlertLevels']);

Route::get('cameras/{id}', [CameraController::class, 'getCamera']);

edited Jun 4, 2021 at 11:11

answered Jun 4, 2021 at 11:03

John Lobo

15.4k2 gold badges18 silver badges25 bronze badges

2 Comments

shaedrich Over a year ago

Good point about encryption. Often times, UUID or GUID is used instead of auto increment integer. Regexes can be defined for that cases as well.

John Lobo Over a year ago

@shaedrich.ya but i am not good at regex so.:)

Collectives™ on Stack Overflow

Laravel API: url matching in api.php file

2 Answers 2

Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related