-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
TakesArbitraryInput fixes #22027
TakesArbitraryInput fixes #22027
Conversation
Update the OpenAPI generation code to render TakesArbitraryInput appropriately. Mark the cubbyhole write operation as TakesArbitraryInput. Contributes to fixing hashicorp/vault-client-go#201. We will also need hashicorp/vault-plugin-secrets-kv#114 merged and a new version of that plugin brought into Vault.
This is a tiny part of general trends toward making the OpenAPI better, so I'm inclined to say just |
Contributes to fixing #201 These template changes make no difference to the code right now, but combined with the incoming OpenAPI changes in hashicorp/vault#22027 and hashicorp/vault-plugin-secrets-kv#114 will effect the fix.
#22028 opened to bring that into Vault. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks a lot for this!
This includes the results of: * hashicorp/vault#21949 * hashicorp/vault#21942 * hashicorp/vault#22027 The general direction of change is pretty positive, but in reviewing the generated diffs, I have spotted an issue: we are now generating correct parameters for query parameters... but those query parameters are being handled as individual function parameters... which are generated in the order listed in the OpenAPI spec... which is dependent on random Go hash iteration ordering. Ugh. Oh well, let's catch up with latest developments in the Vault repo for now, and I'll go put in a new PR to at least sort the parameters alphabetically. Longer term, this is probably going to push us in the direction of excluding the GET version of APIs with equivalent GET and POST versions from the generated libraries.
…150) * Sync OpenAPI; Multiple parameter correctness changes This includes the results of: * hashicorp/vault#21949 * hashicorp/vault#21942 * hashicorp/vault#22027 The general direction of change is pretty positive, but in reviewing the generated diffs, I have spotted an issue: we are now generating correct parameters for query parameters... but those query parameters are being handled as individual function parameters... which are generated in the order listed in the OpenAPI spec... which is dependent on random Go hash iteration ordering. Ugh. Oh well, let's catch up with latest developments in the Vault repo for now, and I'll go put in a new PR to at least sort the parameters alphabetically. Longer term, this is probably going to push us in the direction of excluding the GET version of APIs with equivalent GET and POST versions from the generated libraries. * Update the templates with a missing using directive
Update the OpenAPI generation code to render TakesArbitraryInput
appropriately.
Mark the cubbyhole write operation as TakesArbitraryInput.
Contributes to fixing
hashicorp/vault-client-go#201.
We will also need
hashicorp/vault-plugin-secrets-kv#114 merged and
a new version of that plugin brought into Vault.