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

Update Exception Logging #6671

Merged
merged 3 commits into from
Aug 4, 2023
Merged

Conversation

scbedd
Copy link
Member

@scbedd scbedd commented Aug 3, 2023

Lower the StdErrErrorThreshold to Error so that Logging.Error show up on stderr.

Resolves #6563 so that the .NET framework will actually see the stderr output when collecting it for the error.

@scbedd scbedd requested a review from mikeharder as a code owner August 3, 2023 20:26
@scbedd scbedd self-assigned this Aug 3, 2023
@scbedd scbedd merged commit be8d5b4 into Azure:main Aug 4, 2023
scbedd added a commit that referenced this pull request Aug 21, 2023
This reverts commit be8d5b4.

Generating a version of the proxy WITH the logging fixes from #6812 but
without the fixes to stderr output.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

More diagnostic information needed when recording syncing fails
3 participants