Skip to content
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: #887 allow multiple params with wildcard #898

Merged
merged 2 commits into from
Feb 2, 2024

Conversation

Mitchell3514
Copy link
Contributor

  • add test for paths with multiple params including a wildcard param
  • update the toExpressParams wildcard regex

Fixes: #887

Mitchell3514 and others added 2 commits January 31, 2024 17:42
…ms (#1)

* Change regex to support multiple params when including URI path param
* Update regex, remove unnecessary bracket
@cdimascio cdimascio merged commit 2d33d0a into cdimascio:master Feb 2, 2024
5 checks passed
@cdimascio
Copy link
Owner

Thanks!

@guillerecalde
Copy link
Contributor

Hey @cdimascio, any chance you have an ETA for when this fix will be released?

Copy link
Owner

@cdimascio cdimascio left a comment

Choose a reason for hiding this comment

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

v5.1.4+ contains the fix

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Multiple path params break support for URI path param
3 participants