openapi-to-postman
openapi-to-postman copied to clipboard
postmanlabs
Add the alwaysInheritAuthentication option
This option forces all requests to inherit auth from the top of the collection. This is really useful when all requests should be using the same authentication. If authentication is called out on each request in the Postman app, they will not inherit the options set on the collection, like the token, and must all be set manually. By allowing all requests to simply inherit from the collection, the token can be set once at the top of the collection and all requests will use it automatically.
Fixes #271
@SahilChoudhary22 can I get a review on this? I would very much appreciate this feature. Hopefully it can be reviewed/approved soon!
@AndrewGuenther Sure, planning to review this before releasing the next version 👍
@AndrewGuenther Thanks for your contributions to the module. After reviewing the pull request, I noticed an area where I could use some additional clarity. I've left a comment on that line.
That being said, since the PR is now outdated and now the openapi-to-postman has 2 versions of converter within it, we will need to make some additional changes to add the support for alwaysInheritAuthentication to convertV2, along with the respective test cases. I will be raising another pull request shortly, where I'll be cherry-picking the changes from this PR and the added support for v2. 🙇 Stay tuned!
Hey @SahilChoudhary22, thanks for reviewing this PR. You mentioned that you would be raising another pull request that would supersede this one. Has there been any progress on that front?
Hi @tmccnnll ,
I've raised the PR here - https://github.com/postmanlabs/openapi-to-postman/pull/728
The new option should be available with the next release of the module ie. 4.14.x