81 refactor i uzupełnienie swaggera do metod auth

[JanekDr]

• Migracja klas: Wszystkie endpointy Solvro/USOS OAuth przeniesione na DRF APIView/AsyncAPIView dla poprawnego generowania schematu. Wszelkie opcje redirect korzystają z parametryzacji docelowych widoków klasowych.
• Opisy wejść/wyjść: Dodano dekoratory @extend_schema ze szczegółowym wykazem rzucanych kodów błędów (200, 302, 400, 401, 403, 404) dla każdej ścieżki JWT i OTP.
• Bugfix walidacji na OTP: Dodano authentication_classes = [] do publicznych widoków OTP i LoginLinkView. Eliminacja błędnych response 401 Unauthorized dla starych sesji.
• Wszystkie endpointy są dostępne w sekcji Authentication w dokumentacji.

[Antoni-Czaplicki]

@JanekDr co myślisz żeby zrobić helper safe_redirect który w sobie sprawdzi czy jest safe i jak coś to nadpisze?

[Antoni-Czaplicki]

i pamiętaj też żeby dać Resolve jak już jest komentarz ogarnięty

[JanekDr]

@JanekDr co myślisz żeby zrobić helper safe_redirect który w sobie sprawdzi czy jest safe i jak coś to nadpisze?

Myśle ze to dobry pomysł i uszczupli lekko kod w metodach auth dla usosa i solvro

[Copilot]

Pull request overview

Refactors the authentication-related OAuth and OTP endpoints in users to improve DRF Spectacular (Swagger/OpenAPI) schema generation, while tightening redirect safety checks and documenting request/response variants.

Changes:

• Migrates Solvro/USOS OAuth endpoints from function-based views to DRF APIView / adrf async API views.
• Expands @extend_schema documentation across JWT, OAuth, and OTP endpoints (params, examples, and response codes).
• Updates redirect safety logic and adds tests for DEBUG-dependent HTTP/HTTPS redirect handling; adds initial OAuth flow tests.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.

File	Description
users/views.py	New class-based OAuth endpoints, enhanced schema docs, updated redirect safety logic, and public OTP views made explicitly unauthenticated.
users/urls.py	Routes updated to point to the new class-based OAuth views.
users/tests/test_redirect_url.py	Adds DEBUG=True/False coverage for HTTP vs HTTPS redirect acceptance.
users/tests/test_oauth.py	Adds tests asserting callback URL composition and a mocked Solvro authorize flow.

Comments suppressed due to low confidence (1)

users/views.py:574

• Same add_query_params + view-name issue in the banned-user jwt=true branch: if redirect_url is the default view name, redirect(add_query_params(...)) can raise NoReverseMatch and return 500 instead of redirecting with an error.

            if request.GET.get("jwt", "false") == "true":
                auth_params = {"error": "user_banned"}
                if user.ban_reason:
                    auth_params["ban_reason"] = user.ban_reason
                return redirect(add_query_params(redirect_url, auth_params))

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.