You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Most REST endpoints in the edx-platform that require AUTH support JWTAuth or would if it was a default auth class.
However, some of the endpoints are explicitly set to only use SessionAuth. We should update these endpoints so that instead of explicitly overriding the DRF Authentication Classes they use the default authentication classes.
Tasks
Update all API views that explicitly accept only SessionAuth to use the DEFAULT_AUTHENTICATION_CLASSES instead.
Relevant Views/URLs
/api/bookmarks/v1/bookmarks/ : get
/api/bookmarks/v1/bookmarks/ : post
/api/bookmarks/v1/bookmarks/{username},{usage_id}/ : get
@feanil Just for confirmation that my understanding of this ticket is correct, as per ticket acceptance criteria we have to remove the authentication from view and use the default one if there is just SessionAuth and add the JWT in default authentication so that all endpoints would allow JWT auth?
I found almost the majority of the views using the combination authentication_classes = (SessionAuthentication, JwtAuthentication) or SessionAuthenticationAllowInactiveUser but few are using authentication_classes = (authentication.SessionAuthentication).
@salman2013 mostly correct. I think we just need to update the ones that are using authentication_classes = (authentication.SessionAuthentication) as a part of this ticket. There is already other work in progress to add JWT auth as a default. Once we do that we'll do another pass where we remove overrides for any views where the overrides match the default.
Most REST endpoints in the edx-platform that require AUTH support JWTAuth or would if it was a default auth class.
However, some of the endpoints are explicitly set to only use SessionAuth. We should update these endpoints so that instead of explicitly overriding the DRF Authentication Classes they use the default authentication classes.
Tasks
Relevant Views/URLs
Acceptance Criteria
The text was updated successfully, but these errors were encountered: