Merge pull request #127 from OpenSTEF/bugfix/KTPS-3358-fix-empty-params

Update data_interface.py

.github/workflows/python-upload-package.yml

@@ -11,6 +11,10 @@ on:
 jobs:
   deploy:
     runs-on: ubuntu-latest
+    environment: release
+    permissions:
+      # IMPORTANT: this permission is mandatory for trusted publishing
+      id-token: write
     steps:
     # Checkout
     - name: Checkout code
@@ -26,10 +30,8 @@ jobs:
         python -m pip install --upgrade pip
         pip install setuptools wheel twine
     # Build and publish
-    - name: Build and publish
-      env:
-        TWINE_USERNAME: ${{ secrets.PYPI_USERNAME }}
-        TWINE_PASSWORD: ${{ secrets.PYPI_PASSWORD }}
-      run: |
-        python setup.py sdist bdist_wheel
-        twine upload dist/*
+    - name: Build
+      run: python setup.py sdist bdist_wheel
+    - name: Publish
+      uses: pypa/gh-action-pypi-publish@release/v1
+

openstef_dbc/data_interface.py

@@ -222,8 +222,6 @@ def check_influx_available(self):
         return available
 
     def exec_sql_query(self, query: str, params: dict = None, **kwargs):
-        if params is None:
-            params = {}
         try:
             return pd.read_sql(query, self.mysql_engine, params=params, **kwargs)
         except sqlalchemy.exc.OperationalError as e:
@@ -239,8 +237,6 @@ def exec_sql_query(self, query: str, params: dict = None, **kwargs):
             raise
 
     def exec_sql_write(self, statement: str, params: dict = None) -> None:
-        if params is None:
-            params = {}
         try:
             with self.mysql_engine.connect() as connection:
                 response = connection.execute(statement, params=params)

setup.py

@@ -29,7 +29,7 @@ def read_long_description_from_readme():
 
 setup(
     name="openstef_dbc",
-    version="3.6.19",
+    version="3.6.21",
     packages=find_packages(include=["openstef_dbc", "openstef_dbc.*"]),
     description="Database Connection for OpenSTEF",
     long_description=read_long_description_from_readme(),