Merge pull request #83 from juntossomosmais/fix/database-error

fix(dbapi): remove string interpolation in self.db_info
dynatrace-oss · Aug 23, 2023 · e49966e · e49966e
2 parents 1c26667 + 1d53570
commit e49966e
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 3 deletions.
diff --git a/.gitignore b/.gitignore
@@ -5,3 +5,4 @@ __pycache__
 *.pyc
 .tox/
 autodynatrace.egg-info/
+.idea
diff --git a/autodynatrace/wrappers/dbapi/__init__.py b/autodynatrace/wrappers/dbapi/__init__.py
@@ -8,7 +8,7 @@
 class TracedCursor(wrapt.ObjectProxy):
     def __init__(self, cursor, db_info):
         super(TracedCursor, self).__init__(cursor)
-        self.db_info = f"{db_info}"
+        self.db_info = db_info
         self._self_last_execute_operation = None
         self._original_cursor = cursor
 

diff --git a/autodynatrace/wrappers/psycopg2/wrapper.py b/autodynatrace/wrappers/psycopg2/wrapper.py
@@ -11,7 +11,7 @@ def instrument():
     def parse_dsn(dsn):
         return {c.split("=")[0]: c.split("=")[1] for c in dsn.split() if "=" in c}
 
-    class DynatraceCursor(psycopg2.extra.DictCursorBase):
+    class DynatraceCursor(psycopg2.extensions.cursor):
         def __init__(self, *args, **kwargs):
             self._dynatrace_db_info = kwargs.pop("dynatrace_db_info", None)
             super(DynatraceCursor, self).__init__(*args, **kwargs)

diff --git a/setup.py b/setup.py
@@ -2,7 +2,7 @@
 
 setup(
     name="autodynatrace",
-    version="2.0.0",
+    version="2.0.1",
     packages=find_packages(),
     package_data={"autodynatrace": ["wrappers/*"]},
     install_requires=["wrapt>=1.11.2", "oneagent-sdk>=1.3.0", "six>=1.10.0", "autowrapt>=1.0"],
-Original file line number
+Diff line change
@@ Expand Up / @@ -5,3 +5,4 @@ __pycache__ @@
     *.pyc
     .tox/
     autodynatrace.egg-info/
+    .idea