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

Adding nonce to course clone link #2584

Merged
merged 3 commits into from
Jan 17, 2024
Merged

Adding nonce to course clone link #2584

merged 3 commits into from
Jan 17, 2024

Conversation

brianhogg
Copy link
Contributor

@brianhogg brianhogg commented Jan 15, 2024

Description

Adding nonce to course clone link

How has this been tested?

Manually by visiting the url without the nonce, or with a different nonce key.

Types of changes

Security fix

Checklist:

  • This PR requires and contains at least one changelog file.
  • My code has been tested.
  • My code passes all existing automated tests.
  • My code follows the LifterLMS Coding & Documentation Standards.

@brianhogg brianhogg requested a review from ideadude as a code owner January 15, 2024 20:26
@ideadude ideadude merged commit a7964c4 into dev Jan 17, 2024
7 of 34 checks passed
@ideadude ideadude deleted the fix/clone-csrf branch January 17, 2024 19:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants