Skip to content

Conversation

@uzairr
Copy link
Contributor

@uzairr uzairr commented Jul 10, 2019

LEARNER-3761

Description

Currently, grading endpoints in capa-module are vulnerable as they are not restricted.To avoid it, changes have been made so that they are available only for logged-in users.

@uzairr uzairr requested a review from a team as a code owner July 10, 2019 17:35
@uzairr uzairr requested a review from waheedahmed July 10, 2019 17:35
@uzairr uzairr force-pushed the LEARNER-3761/capa-grading-exception branch from 987d40a to 733a3b4 Compare July 24, 2019 09:55
@uzairr uzairr requested a review from a team July 24, 2019 09:55
@uzairr uzairr force-pushed the LEARNER-3761/capa-grading-exception branch 2 times, most recently from ebf208f to 161683b Compare July 24, 2019 13:47
Copy link
Contributor

@waheedahmed waheedahmed left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Once all tests passes.

Cuurently, grading endpoints in capa module are vulnerable as they
are not restricted.To avoid it, changes have been made so that
they are available only for logged-in users.

LEARNER-3761
@uzairr uzairr force-pushed the LEARNER-3761/capa-grading-exception branch from 161683b to 9276aac Compare July 24, 2019 14:20
@uzairr
Copy link
Contributor Author

uzairr commented Jul 24, 2019

jenkins run bokchoy

@edx-status-bot
Copy link

Your PR has finished running tests. There were no failures.

@uzairr uzairr merged commit 408ca56 into master Jul 25, 2019
@uzairr uzairr deleted the LEARNER-3761/capa-grading-exception branch July 25, 2019 09:23
@edx-pipeline-bot
Copy link
Contributor

EdX Release Notice: This PR has been deployed to the staging environment in preparation for a release to production on Friday, July 26, 2019.

@edx-pipeline-bot
Copy link
Contributor

EdX Release Notice: This PR has been deployed to the production environment.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants