-
-
Notifications
You must be signed in to change notification settings - Fork 213
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
RequestValidator.validate uses req.originalUrl #242
Comments
@jtinbergen thanks. This should get fixed. I like your proposal. {expressRoute: "/v1/pets/:id", openApiRoute: "/v1/pets/{id}", ...} Either seems like a good candidate to replace i'll plan to look at this in the coming days. In the meantime, if you're up to it, feel free to experiment as well. PRs are certainly welcome. |
@jtinbergen this should be fixed in |
This results in a lot of cache entries in the middlewareCache because all the parameter values are taken into consideration too.
/customer/1234/detail/1289
/customer/1235/detail/1288
These two calls with the same method and content type create two entries in the cache. Perhaps the openAPI url definition /customer/{something}/detail/{somthingelse} would be a better key?
The text was updated successfully, but these errors were encountered: