Skip to content

Commit

Permalink
Merge pull request #41 from edx/dsheraz/educator-3926
Browse files Browse the repository at this point in the history
Reorder authentication classes to counter 403 error on iOS app
  • Loading branch information
DawoudSheraz authored Mar 11, 2019
2 parents c24db32 + 618b86f commit 10997a9
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 2 deletions.
1 change: 1 addition & 0 deletions AUTHORS
Original file line number Diff line number Diff line change
Expand Up @@ -10,3 +10,4 @@ Usman Khalid <[email protected]>
Rabia Iftikhar <[email protected]>
Ahsan Ulhaq <[email protected]>
Robert Raposa <[email protected]>
Syed Muhammad Dawoud Sheraz Ali <[email protected]>
4 changes: 4 additions & 0 deletions CHANGELOG.rst
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,10 @@ Change Log
Unreleased
~~~~~~~~~~
[1.0.2] - 2019-03-11
--------------------

* Fix the 403 error occurring for completion-batch API for requests coming from the iOS devices

[1.0.0] - 2018-10-16
--------------------
Expand Down
2 changes: 1 addition & 1 deletion completion/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,6 @@
from __future__ import unicode_literals


__version__ = '1.0.1'
__version__ = '1.0.2'

default_app_config = 'completion.apps.CompletionAppConfig' # pylint: disable=invalid-name
2 changes: 1 addition & 1 deletion completion/api/v1/views.py
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ class CompletionBatchView(APIView):
Handles API requests to submit batch completions.
"""
authentication_classes = (
JwtAuthentication, SessionAuthenticationAllowInactiveUser, OAuth2AuthenticationAllowInactiveUser,
JwtAuthentication, OAuth2AuthenticationAllowInactiveUser, SessionAuthenticationAllowInactiveUser,
)
permission_classes = (permissions.IsAuthenticated, IsStaffOrOwner,)
REQUIRED_KEYS = ['username', 'course_key', 'blocks']
Expand Down

0 comments on commit 10997a9

Please sign in to comment.