Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

source-http-ingest: updating tracing::error statement #2033

Closed
wants to merge 1 commit into from

Conversation

jgraettinger
Copy link
Member

@jgraettinger jgraettinger commented Oct 9, 2024

Use ?debug instead of %display to include the error chain.

Workflow steps:

(How does one use this feature, and how has it changed)

Documentation links affected:

(list any documentation links that you created, or existing ones that you've identified as needing updates, along with a brief description)

Notes for reviewers:

(anything that might help someone review this PR)


This change is Reviewable

Use ?debug instead of %display to include the error chain.
@psFried
Copy link
Member

psFried commented Oct 9, 2024

superseded by #2034

@psFried psFried closed this Oct 9, 2024
@jgraettinger jgraettinger deleted the jgraettinger/source-http-ingest branch October 9, 2024 16:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants