-
-
Notifications
You must be signed in to change notification settings - Fork 4
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
remove api-contract-validator for OpenAPI testing #5
remove api-contract-validator for OpenAPI testing #5
Conversation
This will be moved into its own package to be separately maintained
If someone wants to setup a Feel free to create the repository and I'll base it off the current |
Thanks for the PR. Here you go for the repo. https://github.com/japa/openapi-assertions Can we please make the package structure based off this template https://github.com/thetutlage/pkg-template? Essentially, the starting point could be git clone repo
cd repo
npx giget@latest gh:thetutlage/pkg-template . --force |
This comment was marked as outdated.
This comment was marked as outdated.
@thetutlage I've now the code written for the extracted OpenAPI assertions, as mentioned, I need at least one file in that repository such that I can fork and do a pull request. Perhaps use the
|
Sure. I have just scaffolded the project https://github.com/japa/openapi-assertions |
And here's the most minimal extraction I could do: japa/openapi-assertions#1 There's a few minor changes:
|
Documentation PR: japa/japa.dev#26 |
Thanks @ThisIsMissEm 👍 |
BREAKING CHANGE: API assertions have been moved to its own package
🔗 Linked issue
Per #2, this removes the dependency on api-contract-validator which has numerous security vulnerabilities, along with generally being unmaintained. In an effort to achieve a secure core for Japa, this pull request removes the OpenAPI spec testing functionality and it will be moved into a separate Japa Plugin later.
❓ Type of change
📚 Description
See #2 for the details.
I also noted that several of the assertion messages had changed along with one
ts-expect-error
no longer being necessary.📝 Checklist
Question: where is the documentation to submit a PR to it?