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.

Sign up for GitHub

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

Jump to bottom

CosmosException.Message should include CosmosDiagnostics #2158

Closed
j82w opened this issue Jan 27, 2021 · 1 comment
Closed

CosmosException.Message should include CosmosDiagnostics #2158

j82w opened this issue Jan 27, 2021 · 1 comment
Assignees
@asketagarwal
Labels
Diagnostics Issues around diagnostics and troubleshooting

Comments

@j82w
Copy link
Contributor

j82w commented Jan 27, 2021

Currently the CosmosDiagnostics is only included in the CosmosException.ToString(). The issue is most customer are only recording the Exception.Message and Exception.StackTrace causing the important CosmosDiagnostics information to be missing.
@j82w j82w added the Diagnostics Issues around diagnostics and troubleshooting label Jan 27, 2021
@asketagarwal asketagarwal self-assigned this Feb 1, 2021
@asketagarwal asketagarwal mentioned this issue Feb 10, 2021
Closed
@j82w
Copy link
Contributor Author

j82w commented Oct 13, 2021

Fixed in #2719

@j82w j82w closed this as completed Oct 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@asketagarwal asketagarwal

Labels
Diagnostics Issues around diagnostics and troubleshooting
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

CosmosException: Adds Cosmos Diagnostics to CosmosException.Message Azure/azure-cosmos-dotnet-v3
2 participants
@j82w @asketagarwal