Skip to content
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

Fix Workaround for OpenAPI Generator Issue #52

Closed
telday opened this issue Dec 7, 2020 · 0 comments · Fixed by #156
Closed

Fix Workaround for OpenAPI Generator Issue #52

telday opened this issue Dec 7, 2020 · 0 comments · Fixed by #156

Comments

@telday
Copy link
Contributor

telday commented Dec 7, 2020

Summary

We had to add a workaround for an issue in OpenAPI generator (and partly in Conjur) which caused the request to be sent with a Content-Type: application/json header by default, even with an empty request body.

In order to fix the issue temporarily I added a body to the /host_factory_tokens DELETE method, which tricks the generator into using the text/plain MIME type instead of application/json. This workaround should be removed as soon as either Conjur is updated to handle a request like this properly, or the OpenAPI generator is fixed.

Here is the relevant change in the API spec. The integration test for revoke_host_token will also have to be updated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging a pull request may close this issue.

2 participants