Always fall back to string payload if we can't handle structured logs #701
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.
Fixes #689
Supersedes #689
This is slightly broader in scope than #689. That PR only handled the case where we are unable to marshal the payload to JSON. It could still fail to export for other reasons, such as map keys or values containing invalid UTF-8. It could also fail if we didn't support the ValueType (e.g. ValueTypeSlice, or ValueTypeInt).
This PR changes the logs exporter to always fall back to attempting to send a raw string payload if we are unable to create a structpb.Struct from the body.