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

Add CSRF exempt authentication to the batch_completion api #24

Closed
wants to merge 1 commit into from

Conversation

schenedx
Copy link
Contributor

Description: The PR adds CSRF capability to the Completion_Batch api

JIRA: https://openedx.atlassian.net/browse/LEARNER-5542

Dependencies: Need edx-platform to incorporate this.

Installation instructions: Need to have a version pump for edx-platform

Testing instructions: API post to completion_batch endpoint without CSRF token

Reviewers:

Merge checklist:

  • All reviewers approved
  • CI build is green
  • Version bumped
  • Changelog record added
  • Documentation updated (not only docstrings)
  • Commits are squashed
  • PR author is listed in AUTHORS

Post merge:

  • Create a tag
  • Check new version is pushed to PyPi after tag-triggered build is
    finished.
  • Delete working branch (if not needed anymore)

@pomegranited
Copy link
Contributor

Closing this PR due to inactivity -- feel free to re-open if it's still relevant?

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.

2 participants