Fix bug with vault cli when reading an individual field containing a Printf formatting verb #2109
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ns a printf formatting verb
|
LGTM -- thanks! |
chrishoffman
added a commit
to chrishoffman/vault
that referenced
this pull request
Nov 24, 2016
* upstream/master: Add version sha to server startup output Change current version number changelog++ Don't say mlock is supported on OSX when it isn't. (hashicorp#2120) Change command examples for First Secrets hashicorp#2116 (hashicorp#2117) changelog++ cli: fix bug with 'vault read -field=...' when the field value contains a printf formatting verb (hashicorp#2109) Update docs to fix hashicorp#2102 Updating changelog http: increase request limit from 8MB to 32MB http: limit maximum request size Update libraries doc for Haskell community library (hashicorp#2101) Don't exclude 0 from the set of valid polynomials in Shamir. This leads to a potential (although extremely trivial) amount of information leakage. Bump proto files after update changelog++ check for failure on that mysql query (hashicorp#2105) Bump deps Document bug causing certain LDAP settings to be forgotten on upgrade to 0.6.1+. Set number of pester retries to zero by default and make seal command… (hashicorp#2093)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There appears to be an interesting bug with the Vault CLI tool with the
-fieldoption. If the selected field's value contains a formatting verb, e.g.:%s, the output of the CLI tool will contain the familiar%!s(MISSING)in the middle of the value.The fix is simply printing withFprintinstead ofFprintf(which appears to be unnecessary anyway).Edit: Simplified this even further. The Sprintf seems superfluous. Eliminating it and passing the format string to Fprintf fixes this.
Steps to reproduce, if interested: