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

Support for URI path params #457

Closed
CristianUser opened this issue Nov 13, 2020 · 4 comments
Closed

Support for URI path params #457

CristianUser opened this issue Nov 13, 2020 · 4 comments
Assignees
Labels
enhancement New feature or request

Comments

@CristianUser
Copy link

**Is your feature request related to a problem? **
I have an assets server and I use /file/* to take the rest as file location example /file/folder/image.png. following RFC 6570 I should be able to do it with /file{/path*} but it seems to be invalid here

Describe the solution you'd like
Add support to Path segment templates, if there's a way to do it currently please let me know

@cdimascio cdimascio added the enhancement New feature or request label Nov 28, 2020
@cdimascio
Copy link
Owner

@CristianUser OpenAPI 3.x does not support RFC 6570, hence neither does the validator.
I happen to have the same use case. Currently, I use ignorePaths to ignore the route that requires this, then i validate it manually. That said, I'm experimenting with wildcards that are syntactically compatible with OpenAPI. We'll see where this goes. Ideally, the spec would support RFC 6570. If I come up with somethigng, I'd love to get your feedback.

@cdimascio
Copy link
Owner

@CristianUser have a look at #491
Checkout the README update for this PR. It provides a description of the feature.

Unfortunately, it does not conform to RFC-6570. I'll certainly add support of OpenAPI supports it. In the meantime, the PR will satisfy a common case (which i believe is the case you are looking for)

Please share your thoughts

@cdimascio cdimascio self-assigned this Dec 16, 2020
@CristianUser
Copy link
Author

@CristianUser have a look at #491
Checkout the README update for this PR. It provides a description of the feature.

Unfortunately, it does not conform to RFC-6570. I'll certainly add support of OpenAPI supports it. In the meantime, the PR will satisfy a common case (which i believe is the case you are looking for)

Please share your thoughts

I like it, it should work for me 😅

@cdimascio
Copy link
Owner

@CristianUser this is available in v.4.9.1

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

No branches or pull requests

2 participants