-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Do not report exception message in fault details #73034
Conversation
// If we couldn't get a stack, do this | ||
return exception.Message; | ||
// If we couldn't get a stack, report a generic message. | ||
// The exception message is already reported in a separate cred-scanned property. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the fault description field is not client side cred-scanned by VS telemetry. We should not report the message again here, there is already an exception message property that is cred-scanned.
@@ -78,10 +78,9 @@ internal static bool ResetRenameTrackingStateWorker(Workspace workspace, Documen | |||
if (textBuffer == null) | |||
{ | |||
var ex = new InvalidOperationException(string.Format( | |||
"document with name {0} is open but textBuffer is null. Textcontainer is of type {1}. SourceText is: {2}", | |||
"document with name {0} is open but textBuffer is null. Textcontainer is of type {1}.", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This isn't 100% necessary to resolve the bug given we also have the change below (and assuming the message field is cred scanned), but in general we shouldn't be reporting the full source in fault telemetry.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
First part of fixes for https://devdiv.visualstudio.com/DevDiv/_workitems/edit/2032081