-
Notifications
You must be signed in to change notification settings - Fork 433
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
[Port dspace-7_x] Ensures CSRF token is initialized prior to first modifying (non-GET) request #3063
[Port dspace-7_x] Ensures CSRF token is initialized prior to first modifying (non-GET) request #3063
Conversation
NOTE: The e2e test failures in this PR are the result of the Backend PR (DSpace/DSpace#9599) not yet being merged. I've verified locally (via |
👍 I've tested this locally alongside DSpace/DSpace#9599 and verified that this functionality is successfully backported. Namely:
|
Hi @tdonohue, |
…e". Fixes random failures starting e2e test backend via Docker
b88785b
to
2d81a48
Compare
I rebased this on |
Briefly closing & reopening to trigger tests to run again. They should now succeed since the backend PR (DSpace/DSpace#9599) was merged |
Merging immediately as all tests pass, and was tested by @kshepherd as described in DSpace/DSpace#9599 (review) |
Description
GET
) request #2886 todspace-7_x
/csrf
endpoint to allow for forcing refresh of CSRF token DSpace#9599Instructions for Reviewers
/csrf
endpoint to allow for forcing refresh of CSRF token DSpace#9599