Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Added ErrorHandler and ErrorHandlerWithContext in CSRF middleware #2257

Merged
merged 5 commits into from
Sep 1, 2022
Merged

Added ErrorHandler and ErrorHandlerWithContext in CSRF middleware #2257

merged 5 commits into from
Sep 1, 2022

Conversation

mojixcoder
Copy link
Contributor

Fixes #2183

@codecov
Copy link

codecov bot commented Sep 1, 2022

Codecov Report

Merging #2257 (fc8a542) into master (534bbb8) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #2257      +/-   ##
==========================================
+ Coverage   92.40%   92.41%   +0.01%     
==========================================
  Files          37       37              
  Lines        4423     4429       +6     
==========================================
+ Hits         4087     4093       +6     
  Misses        245      245              
  Partials       91       91              
Impacted Files Coverage Δ
middleware/csrf.go 89.42% <100.00%> (+0.64%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Contributor

@aldas aldas left a comment

Choose a reason for hiding this comment

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

LGTM!

Thank you

@aldas aldas merged commit d77e8c0 into labstack:master Sep 1, 2022
@aldas aldas mentioned this pull request Sep 4, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Custom CSRF ErrorHandler
2 participants