Commit graph

4 commits

Author SHA1 Message Date
KoenigMjr 4415697d31 (bugfix/mysql): Change: Remove whitespace and check for empty strings
Error:

??? python[22805]: Traceback (most recent call last):
??? python[22805]:    File "/opt/boswatch3/venv/lib/python3.13/site-packages/mysql/connector/connection_cext.py", line 772, in cmd_query
??? python[22805]:       self._cmysql.query(
??? python[22805]:       ~~~~~~~~~~~~~~~~~~^
??? python[22805]:             query,
??? python[22805]:             ^^^^^^
??? python[22805]:       ...<3 lines>...
??? python[22805]:             query_attrs=self.query_attrs,
??? python[22805]:             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
??? python[22805]:       )
??? python[22805]:       ^
??? python[22805]: _mysql_connector.MySQLInterfaceError: Query was empty
??? python[22805]: The above exception was the direct cause of the following exception:
??? python[22805]: Traceback (most recent call last):
??? python[22805]:    File "/opt/boswatch3/bw_server.py", line 79, in <module>
??? python[22805]:       if not bwRoutMan.buildRouters(bwConfig):
??? python[22805]:              ~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^
??? python[22805]:    File "/opt/boswatch3/boswatch/router/routerManager.py", line 77, in buildRouters
??? python[22805]:       loadedClass = importedFile.BoswatchPlugin(routeConfig)
??? python[22805]:    File "/opt/boswatch3/plugin/mysql.py", line 35, in __init__
??? python[22805]:       super().__init__(__name__, config)  # you can access the config class on 'self.config'
??? python[22805]:       ~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^
??? python[22805]:    File "/opt/boswatch3/plugin/pluginBase.py", line 54, in __init__
??? python[22805]:       self.onLoad()
??? python[22805]:       ~~~~~~~~~~~^^
??? python[22805]:    File "/opt/boswatch3/plugin/mysql.py", line 60, in onLoad
??? python[22805]:       self.cursor.execute(stmnt)
??? python[22805]:       ~~~~~~~~~~~~~~~~~~~^^^^^^^
??? python[22805]:    File "/opt/boswatch3/venv/lib/python3.13/site-packages/mysql/connector/cursor_cext.py", line 353, in execute
??? python[22805]:       self._connection.cmd_query(
??? python[22805]:       ~~~~~~~~~~~~~~~~~~~~~~~~~~^
??? python[22805]:             self._stmt_partition["mappable_stmt"],
??? python[22805]:             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
??? python[22805]:       ...<2 lines>...
??? python[22805]:             raw_as_string=self._raw_as_string,
??? python[22805]:             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
??? python[22805]:       )
??? python[22805]:       ^
??? python[22805]:    File "/opt/boswatch3/venv/lib/python3.13/site-packages/mysql/connector/opentelemetry/context_propagation.py", line 97, in wrapper
??? python[22805]:       return method(cnx, *args, **kwargs)
??? python[22805]:    File "/opt/boswatch3/venv/lib/python3.13/site-packages/mysql/connector/connection_cext.py", line 781, in cmd_query
??? python[22805]:       raise get_mysql_exception(
??? python[22805]:             err.errno, msg=err.msg, sqlstate=err.sqlstate
??? python[22805]:       ) from err
??? python[22805]: mysql.connector.errors.ProgrammingError: 1065 (42000): Query was empty
2025-11-21 17:42:38 +01:00
Luflosi d4dcc75711
Avoid "DeprecationWarning: invalid escape sequence"
Without this change, many warnings like this will be generated while running pytest:
```
test/test_template.py:3
  /build/source/test/test_template.py:3: DeprecationWarning: invalid escape sequence '\/'
    """!
```
This can also be seen when manually running python with warnings enabled.

This happens because the comment uses a multiline string and Python interprets the backslash in the logo as an escape character and complains that \/ is not a valid escape sequence. To fix this, prepend the string with the letter r to indicate that the backslash should be treated as a literal character, see https://docs.python.org/3/reference/lexical_analysis.html#index-20.
I also applied this change to all the comment strings since that shouldn't break anything and to establish it as a pattern for the future so this problem hopefully never happens again.

This is what I did specifically:
- Change the comment at the top of bw_client.py and bw_server.py to start with `"""!` since that seems to be the pattern here
- Search-and-Replace all occurances of `"""!` with `r"""!`
- Manually change the strings in `logoToLog()` in boswatch/utils/header.py
2023-09-19 17:49:09 +02:00
Jan Speller 90d932d53c update mysql plugin: add init_db.sql and create table automatically 2021-02-28 13:58:29 +01:00
Jan Speller 9d73be593f implement basic mysql functionality 2021-02-16 01:09:04 +01:00