-
-
Notifications
You must be signed in to change notification settings - Fork 50
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
fix: Prevent response storage calls if method is not supported #464
Conversation
Codecov Report
@@ Coverage Diff @@
## main #464 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 17 17
Lines 507 516 +9
Branches 155 159 +4
=========================================
+ Hits 507 516 +9
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
Awesome!! Can you add a test case for it? |
I added tests for fulfill and my new change regarding reject. Please feel free to point out any needed changes. :) |
Thank you so much for your catch... I'll release ASAIC. |
It seems plugin performs a
storage.get
call when getting response from requests that do not meet the method criteria (e.g. method POST).By following the same principle in request interceptor, we make sure response does not make unnecessary calls.