You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
With the new feature implemented in #33 which we appreciate being implemented! :)
I expected to solve our use case by adding 2 instances of the Middleware, but that does not work.
Ex.
Instance 1 ( We wan't to retry on all 429 no matter the method):
The issue(from what I can tell) is that the "combined" options created as the first thing inside __invoke from the middleware is provided to $next (the second middleware), which then overwrites the default options (The second set of rules for the other middleware).
The text was updated successfully, but these errors were encountered:
Detailed description
With the new feature implemented in #33 which we appreciate being implemented! :)
I expected to solve our use case by adding 2 instances of the Middleware, but that does not work.
Ex.
Instance 1 ( We wan't to retry on all 429 no matter the method):
Instance 2 (Retry on all GET requests for another set of response codes):
The issue(from what I can tell) is that the "combined" options created as the first thing inside
__invoke
from the middleware is provided to$next
(the second middleware), which then overwrites the default options (The second set of rules for the other middleware).The text was updated successfully, but these errors were encountered: