-
Notifications
You must be signed in to change notification settings - Fork 597
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
dbapi instrumentation breaks psycopg2 when registering types. #1077
Labels
bug
Something isn't working
Comments
ocelotl
added a commit
to ocelotl/opentelemetry-python-contrib
that referenced
this issue
May 19, 2022
ocelotl
added a commit
to ocelotl/opentelemetry-python-contrib
that referenced
this issue
May 19, 2022
ocelotl
added a commit
to ocelotl/opentelemetry-python-contrib
that referenced
this issue
May 23, 2022
ocelotl
added a commit
to ocelotl/opentelemetry-python-contrib
that referenced
this issue
May 24, 2022
ocelotl
added a commit
to ocelotl/opentelemetry-python-contrib
that referenced
this issue
May 24, 2022
ocelotl
added a commit
to ocelotl/opentelemetry-python-contrib
that referenced
this issue
Jun 6, 2022
ocelotl
added a commit
that referenced
this issue
Jun 7, 2022
* Refactor TracedConnectionProxy Fixes #1077 * Remove unecessary methods
Closed
If the fix for this has been reverted, should the issue be reopened? Or is being tracked elsewhere? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe your environment
I'm using dbapi instrumentation to instrument postgresql. When I attempt to register jsonb, psycopg2 crashes. This issue is nearly identical to #143, with the exception that I'm not using
Psycopg2Instrumentor
.The reason I'm using dbapi instrumentation vs
Psycopg2Instrumentor
is because the latter does not support capturing parameters as span attributes.Steps to reproduce
results in
What is the expected behavior?
For psycopg2 to not crash when used with dbapi instrumentation.
What is the actual behavior?
It crashes when psycopg2 enforces connection or cursor types.
Additional context
Add any other context about the problem here.
The text was updated successfully, but these errors were encountered: