From e47a5fdf061ba5e12f6d4582489b502b4ff516ac Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Malthe=20J=C3=B8rgensen?= <malthe.jorgensen@gmail.com>
Date: Sun, 31 Mar 2019 23:04:36 +0200
Subject: [PATCH] Allow omitting port for Postgres and MySQL

Allows omitting the port in MB_DB_CONNECTION_URI when using Postgres and
MySQL.

Defaults to port 5432 for Postgres and 3306 for MySQL.
---
 src/metabase/db/spec.clj | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/metabase/db/spec.clj b/src/metabase/db/spec.clj
index 7c1fd20fc37..fc16f0d398f 100644
--- a/src/metabase/db/spec.clj
+++ b/src/metabase/db/spec.clj
@@ -24,7 +24,7 @@
   (merge
    {:classname                     "org.postgresql.Driver"
     :subprotocol                   "postgresql"
-    :subname                       (make-subname host port db)
+    :subname                       (make-subname host (or port 5432) db)
     ;; I think this is done to prevent conflicts with redshift driver registering itself to handle postgres://
     :OpenSourceSubProtocolOverride true}
    (dissoc opts :host :port :db)))
@@ -37,7 +37,7 @@
   (merge
    {:classname   "org.mariadb.jdbc.Driver"
     :subprotocol "mysql"
-    :subname     (make-subname host port db)}
+    :subname     (make-subname host (or port 3306) db)}
    (dissoc opts :host :port :db)))
 
 
-- 
GitLab