openapi-merge icon indicating copy to clipboard operation
openapi-merge copied to clipboard

include and exclude based on paths

Open RevanthGovindan opened this issue 2 years ago • 6 comments

Added configs for include or exclude specification paths based on config provided

RevanthGovindan avatar Apr 04 '22 15:04 RevanthGovindan

Hi @RevanthGovindan , thanks for the PR submission for this feature request. Can you let me know why the includeTags and excludeTags feature does not work for your use case?

robertmassaioli avatar Apr 04 '22 22:04 robertmassaioli

@robertmassaioli includeTags and excludeTags works well, consider if someone wants to exclude or include any specific API, we may need this feature.

RevanthGovindan avatar Apr 05 '22 03:04 RevanthGovindan

@RevanthGovindan If somebody only wants to include/exclude a specific API then they can just tag/untag APIs that way. I'm still not understanding why we need an extra feature to accomplish the same thing? Could you expand on your use case a little more?

robertmassaioli avatar Apr 05 '22 03:04 robertmassaioli

@robertmassaioli consider if someone wants to ignore API in Included tag, so this might be helpful.

RevanthGovindan avatar Apr 05 '22 04:04 RevanthGovindan

@robertmassaioli I have two different microservices, admin and customer module which has same tag called Login and it has API's as below, here Login tag is common for both microservices, but only login API is common i want to ignore account/get-customer from admin module, so this addition could be helpful to filter that. Admin Login:

  1. account/login
  2. account/get-customer

Customer Login:

  1. account/get-details
  2. account/get-bills

RevanthGovindan avatar Apr 05 '22 04:04 RevanthGovindan

This could be really usefull for me too ! Exclude by (path name, method) couple by configuration let's our openapi generation without tag customisation.

Fimeo avatar May 25 '22 14:05 Fimeo