jsonclient: log client URI when logging errors #617
Merged
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.
When dependent software manages submission to several logs by creating multiple
LogClient
instances the error messages logged by the underlyingJSONClient
need to identify theURI
at fault or they don't have enough context to stand-alone. E.g. If aLogClient
fails an initial submission to a log but will retry after a period a line like the following is logged:Without more context it's difficult to determine which log returned a 503 prompting a backed-off retry.
Prior to this PR
JSONClient.PostAndParseWithRetry
logged the underlying URI in one error logging site but not two others, including the bad status code case shared above. This PR adds the URI to the other two sites and adjusts the wording of the existing site to be consistent.Checklist