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

Expose middleware.CreateExtractors function so we can use it from echo-contrib repository #2338

Merged
merged 1 commit into from
Nov 12, 2022

Conversation

aldas
Copy link
Contributor

@aldas aldas commented Nov 12, 2022

Expose middleware.CreateExtractors function so we can use it from echo-contrib repository. JWT middleware will move there.

@codecov
Copy link

codecov bot commented Nov 12, 2022

Codecov Report

Base: 92.37% // Head: 92.37% // Increases project coverage by +0.00% 🎉

Coverage data is based on head (2bde541) compared to base (b010b69).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2338   +/-   ##
=======================================
  Coverage   92.37%   92.37%           
=======================================
  Files          37       37           
  Lines        4446     4448    +2     
=======================================
+ Hits         4107     4109    +2     
  Misses        247      247           
  Partials       92       92           
Impacted Files Coverage Δ
middleware/csrf.go 89.42% <100.00%> (ø)
middleware/extractor.go 99.13% <100.00%> (+0.01%) ⬆️

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

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@aldas aldas merged commit 35184a8 into labstack:master Nov 12, 2022
@aldas aldas mentioned this pull request Dec 27, 2022
@aldas aldas deleted the expose_createExtractors branch December 29, 2022 14:29
# 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.

1 participant