Hey @snocorp, apologies for the silence on this, and thanks for the contribution. I know we were a bit worried about breaking the restriction on GET only calls, but given that the call here is about retrieving credentials and not about changing Vault's config, it should be fine.

I'll give the code a look over and hopefully we can get this in sooner rather than later!

@kpcraig Thanks so much for reviewing and merging! I noticed the build is failing after merging and I believe we need to update the docker compose command. I've created #567 to address this.

@kpcraig Thanks for reviewing this one. I don't see the related changes to dist/index.js in this PR so we will need to make sure to make that change before we do the release.

@fairclothjm Thanks, I didn't realize that was part of the PR process but in retrospect it makes sense. Would it be helpful if I created a new PR for that? I'd love to get a new release so we can start to use it internally.

@kpcraig @fairclothjm I've created #568 to update the build output in case that helps to get the next release out.