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

Compatible with openApi 3.1 #44

Open
p-pichet opened this issue Dec 17, 2024 · 3 comments
Open

Compatible with openApi 3.1 #44

p-pichet opened this issue Dec 17, 2024 · 3 comments

Comments

@p-pichet
Copy link

Hi. I know the bug is not on your side.

The bug come for the package OpenApiPSR7Validator.

I open a PR for it but, for what I can see, there is no more activity on it.

Do you plan to change the package for another one like opis/json-schema ?

@osteel
Copy link
Owner

osteel commented Dec 17, 2024

Salut Philippe,

I have no plan to change the underlying library no, nor do I have the bandwidth to look into it, if I'm honest.

Plus, OpenApiPSR7Validator currently does a lot of the heavy lifting, while from what I gathered after a very quick look at the library you've suggested, it would be quite a bit of work to adapt it to the particular purpose of validating PSR7 messages

@p-pichet
Copy link
Author

Hi.

Thanks for the response.
What is the best for you between :

  • Add a fork of the Validator to update to be compliant with OpenApi 3.1 and use it instead
  • I see to work on the migration to the other library

@osteel
Copy link
Owner

osteel commented Dec 19, 2024

I reckon the latter, since otherwise you go from maintaining a single library to maintaining a library + a fork ^^

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

No branches or pull requests

2 participants