fix: Warn about CLI extensions in OpenAPI spec#317
Conversation
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
|
Thanks again for opening this and sketching the direction. For v2, I ended up taking the smaller visibility-only approach we discussed: no blocking trust prompt, but / now summarize behavior-changing extensions and shows the detailed affected commands and parameters. I opened the v2 implementation in #348 and credited this PR there, so I’m closing this older draft as superseded. |
|
Thanks again for opening this and sketching the direction. For v2, I ended up taking the smaller visibility-only approach we discussed: no blocking trust prompt, but api connect / api sync now summarize behavior-changing x-cli extensions and doctor api shows the detailed affected commands and parameters. I opened the v2 implementation in #348 and credited this PR there, so this older draft is superseded. |
The
x-cli-*OpenAPI specification extensions recognized by Restish can be abused. To prevent this, this change:x-cli-*extensions found in an OpenAPI specificationThis behavior can be disabled by passing the global
--blindly-accept-cli-extensionsflag.Addresses #316.