Skip to content

feat: add option to strip CSRF token from form values after validation - #217

Open
AdamMagued wants to merge 1 commit into
gorilla:mainfrom
AdamMagued:fix-issue-203
Open

AdamMagued wants to merge 1 commit into
gorilla:mainfrom
AdamMagued:fix-issue-203

Conversation

@AdamMagued

Copy link
Copy Markdown

Fixes #203

Summary

  • Added support for stripping the validated CSRF form field from parsed form values so strict form decoders such as gorilla/schema work without requiring ignored-field workarounds.
  • Added unit tests verifying form token cleanup after validation.

Verified via container sandbox test suite.

Signed-off-by: AdamMagued <adamismailmageud@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE] Make csrf and gorilla/schema compatible with each other without additional config

1 participant