SNOW-3379021: Mistaken regex in cursor.py

Python version

All versions

Operating system and processor architecture

All operating systems and processor architectures

Installed packages

N/A

What did you do?

N/A

What did you expect to see?

Both snowflake.connector.cursor and snowflake.connector.aio._cursor have a check like this:

            if re.search(";/s*$", command) is None:
                command = command + "; "

The /s should probably be \s. As-is, this sticks in an extra ;

In [14]: command = 'SELECT * FROM data; '
    ...: if re.search(";/s*$", command) is None:
    ...:                 command = command + "; "
    ...: print(command)
SELECT * FROM data; ;

Contrast with this:

In [15]: command = 'SELECT * FROM data; '
    ...: if re.search(r";\s*$", command) is None:
    ...:                 command = command + "; "
    ...: print(command)
SELECT * FROM data;

Alternatively, this would do the same trick without re:

            if not command.rstrip().endswith(";"):
                command = command + "; "