fastify / fastify-swagger

Swagger documentation generator for Fastify
MIT License
910 stars 200 forks source link

Hide HEAD routes by default #724

Closed mcollina closed 1 year ago

mcollina commented 1 year ago

This PR partially reverts https://github.com/fastify/fastify-swagger/pull/649 due to some unintended breakage. I had a couple of applications whose tests failed badly due to the HEAD routes now included in the definitions. This PR hides by default but add an option to expose them, both at the plugin level or at the route level. The plugin would rely anymore on the Fastify own settings.

Fixes #722

Checklist

mcollina commented 1 year ago

I would like to land this ASAP to avoid more breakages.