This repository has been archived by the owner on Apr 3, 2023. It is now read-only.
Error 404 on route GET users/{id} when not logged in #30
Labels
Bug 🐛
Something isn't working or don't work correctly
Description
When you make a request on route
GET users/{id}
, API returns an error 404 (Resource not found), even if user was found. It should return an error 401 (Unauthorized), both in the case the user does not exist and in the case it does existThere is a problem that occurs in the same situation when doing tests :
However, the problem is not the same, this is an error, and the problem I described earlier was a bug
Investigation of the issue
I tried changing function
supports
of classApp\DataProvider\UserDataVisibilityItemDataProvider
to make it also check for login :$checkLogin = $this->security->getUser() != null;
.This fixes the issue in tests :
This fix does not change anything in a normal call. It seems like by changing this function, the
getItem
method of the same class is not called anymore (which is the behaviour we would expect). But APIPlateform still returns a 404 code instead of a 401 codeThe text was updated successfully, but these errors were encountered: