Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Zniesienie wymogu hasła w users_partial_update #719

Open
ad-m opened this issue Dec 22, 2020 · 3 comments
Open

Zniesienie wymogu hasła w users_partial_update #719

ad-m opened this issue Dec 22, 2020 · 3 comments
Assignees

Comments

@ad-m
Copy link
Member

ad-m commented Dec 22, 2020

obraz

W celu zarządzania użytkownikami powinniśmy znieść wymóg podania hasła użytkownika i nazwy użytkownika przed jego edycją. Uwierzytelnianie jest na innej warstwie. Edycja dowolnego pola użytkownika powinna być możliwa niezależnie.

@rwakulszowa
Copy link
Member

Mogę spróbować się tym zająć.

@rwakulszowa
Copy link
Member

Drobne pytanie (a właściwie dwa):

  1. Skąd pochodzi zrzut powyżej? Nie znalazłem dokumentacji o podobnym wyglądzie.
  2. W którym miejscu dodane jest wymaganie pola password? Z tego co widzę, aktualizacja pojedynczych pól jest obsługiwana przez backend, przy użyciu zapytania PATCH:

curl --user admin:admin http://localhost:8000/api/users/1/ -X PATCH --data '{"firstName": "first_name"}' -H "content-type: application/json"

Zgodnie z dokumentacją (i kodem, na tyle na ile przejrzałem wygenerowany kod SDK), SDK wysyła PATCH:
image

Błąd informujący o braku wymaganego pola password zwracany jest faktycznie przy zapytaniu PUT:

curl --user admin:admin http://localhost:8000/api/users/1/ -X PUT --data '{"firstName": "first_name2"}' -H "content-type: application/json"

{"password":["This field is required."],"username":["This field is required."]}

Co w jakimś stopniu sugeruje, że być może wykonywane jest błędne zapytanie.

@MichalKarol - mogę prosić o więcej szczegółow? Czy problem pojawia się także przy zapytaniu PATCH?

@ad-m
Copy link
Member Author

ad-m commented Dec 31, 2020

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants