Skip to content

Adding diagnostics article#1286

Merged
samsp-msft merged 6 commits into
mainfrom
samsp/diagnosing-yarp
Oct 28, 2021
Merged

Adding diagnostics article#1286
samsp-msft merged 6 commits into
mainfrom
samsp/diagnosing-yarp

Conversation

@samsp-msft
Copy link
Copy Markdown
Contributor

Fixes #660

Comment thread docs/docfx/articles/diagnosing-yarp-issues.md Outdated
Comment thread docs/docfx/articles/diagnosing-yarp-issues.md Outdated
Comment thread docs/docfx/articles/diagnosing-yarp-issues.md Outdated
Comment thread docs/docfx/articles/diagnosing-yarp-issues.md Outdated
Comment thread docs/docfx/articles/diagnosing-yarp-issues.md Outdated
Comment thread docs/docfx/articles/diagnosing-yarp-issues.md Outdated
Copy link
Copy Markdown
Member

@Tratcher Tratcher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What about debugging outgoing requests & HttpClient?

Comment thread docs/docfx/articles/diagnosing-yarp-issues.md Outdated
Comment thread docs/docfx/articles/diagnosing-yarp-issues.md Outdated
Comment thread docs/docfx/articles/diagnosing-yarp-issues.md Outdated
Comment thread docs/docfx/articles/diagnosing-yarp-issues.md Outdated
Comment thread docs/docfx/articles/diagnosing-yarp-issues.md Outdated
Comment thread docs/docfx/articles/diagnosing-yarp-issues.md Outdated
Comment thread docs/docfx/articles/diagnosing-yarp-issues.md Outdated
@Tratcher
Copy link
Copy Markdown
Member

Did this logging get expanded?
#660 (comment)

@samsp-msft
Copy link
Copy Markdown
Contributor Author

Did this logging get expanded? #660 (comment)

A few more details in the completion message wouldn't go amiss, but that can be tackled independently.

Comment thread docs/docfx/articles/diagnosing-yarp-issues.md
Comment thread docs/docfx/articles/diagnosing-yarp-issues.md Outdated
@Tratcher
Copy link
Copy Markdown
Member

Did this logging get expanded? #660 (comment)

A few more details in the completion message wouldn't go amiss, but that can be tackled independently.

Ok, either keep #660 open when you're done here or file a new issue please.

@samsp-msft samsp-msft requested a review from Tratcher October 27, 2021 05:57
Comment thread docs/docfx/articles/diagnosing-yarp-issues.md
Comment thread docs/docfx/articles/diagnosing-yarp-issues.md
Comment thread docs/docfx/articles/diagnosing-yarp-issues.md
@samsp-msft samsp-msft merged commit c222ce8 into main Oct 28, 2021
@samsp-msft samsp-msft deleted the samsp/diagnosing-yarp branch October 28, 2021 19:21
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.

Proxy supplies logs with relevant info for each stage of the request

3 participants