vault-clj
vault-clj copied to clipboard
Specs for authenticate (and other APIs)?
Hello,
What do you think about adding clojure.spec.alpha specs to authenticate and possibly other parts of the public API, similar to how cognitect's AWS client validates requests before sending them?
I wouldn't mind PRing it
Thanks, Ben
I'm generally in favor of more validation, as long as it doesn't get in the way of compatibility with different versions of the vault API.
My idea in particular was to spec authenticate! itself https://github.com/amperity/vault-clj/blob/master/src/vault/core.clj#L12
I'm open to further validation of the requests on APIs, however I don't think we can use clojure.spec and retain babashka compatibility. For example, in ac8a864b0277751b436367aa90160257ed64efce I switched to a really simplified validation approach for the auth and lease states.