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

Fix/sql client db name #2418

Merged
merged 38 commits into from
Jun 12, 2023
Merged

Fix/sql client db name #2418

merged 38 commits into from
Jun 12, 2023

Conversation

jamescrosswell
Copy link
Collaborator

@jamescrosswell jamescrosswell commented Jun 9, 2023

Follow up on Fix/DB Connection spans presented poorly #2409.

Ensures the db.name is recorded for Query/Command spans when using SqlClient (the same way it works for Entity Framework).

jamescrosswell and others added 30 commits May 29, 2023 19:48
Co-authored-by: Matt Johnson-Pint <matt.johnson-pint@sentry.io>
@jamescrosswell jamescrosswell marked this pull request as ready for review June 9, 2023 04:32
@mattjohnsonpint mattjohnsonpint merged commit 333163f into main Jun 12, 2023
@mattjohnsonpint mattjohnsonpint deleted the fix/sql-client-db-name branch June 12, 2023 16:08
@github-actions
Copy link
Contributor

Fails
🚫 Please consider adding a changelog entry for the next release.

Instructions and example for changelog

Please add an entry to CHANGELOG.md to the "Unreleased" section. Make sure the entry includes this PR's number.

Example:

## Unreleased

- Fix/sql client db name ([#2418](https://github.com/getsentry/sentry-dotnet/pull/2418))

If none of the above apply, you can opt out of this check by adding #skip-changelog to the PR description.

Generated by 🚫 dangerJS against 006c404

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