Merge pull request #1151 from fabzzap/main

Do not ignore the socket passed with the -S option, even when no port…
dbcli · Mar 25, 2024 · 707871a · 707871a
2 parents 2f7ea28 + d59cb2a
commit 707871a
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 0 deletions.
diff --git a/changelog.md b/changelog.md
@@ -6,6 +6,7 @@ Bug Fixes:
 ----------
 
 * Don't install tests.
+* Do not ignore the socket passed with the -S option, even when no port is passed
 * Fix unexpected exception when using dsn without username & password (Thanks: [Will Wang])
 * Let the `--prompt` option act normally with its predefined default value
 

diff --git a/mycli/AUTHORS b/mycli/AUTHORS
@@ -31,6 +31,7 @@ Contributors:
   * Daniel Black
   * Daniel West
   * Daniël van Eeden
+  * Fabrizio Gennari
   * François Pietka
   * Frederic Aoustin
   * Georgy Frolov

diff --git a/mycli/main.py b/mycli/main.py
@@ -428,6 +428,7 @@ def connect(self, database='', user='', passwd='', host='', port='',
             port = 3306
             if not host or host == 'localhost':
                 socket = (
+                    socket or
                     cnf['socket'] or
                     cnf['default_socket'] or
                     guess_socket_location()