Merge pull request #10 from gouline/read-only-connection

Read only connection
AlexR2D2 · Feb 8, 2023 · ee885c5 · ee885c5
2 parents 66d4665 + d175c90
commit ee885c5
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/resources/metabase-plugin.yaml b/resources/metabase-plugin.yaml
@@ -15,6 +15,10 @@ driver:
       display-name: Database file
       placeholder: /home/you/the.duckdb (or empty for 'in memory' mode)
       required: false
+    - name: read_only
+      display-name: Establish a read-only connection
+      default: false
+      type: boolean
 init:
   - step: load-namespace
     namespace: metabase.driver.duckdb

diff --git a/src/metabase/driver/duckdb.clj b/src/metabase/driver/duckdb.clj
@@ -14,12 +14,13 @@
 (driver/register! :duckdb, :parent :sql-jdbc)
 
 (defmethod sql-jdbc.conn/connection-details->spec :duckdb
-  [_ {:keys [database_file], :as details}]
+  [_ {:keys [database_file, read_only], :as details}]
   (let [conn_details (merge
-   {:classname   "org.duckdb.DuckDBDriver"
-    :subprotocol "duckdb"
-    :subname     (or database_file "")}
-   (dissoc details :database_file :port :engine))]
+   {:classname         "org.duckdb.DuckDBDriver"
+    :subprotocol       "duckdb"
+    :subname           (or database_file "")
+    "duckdb.read_only" (str read_only)}
+   (dissoc details :database_file :read_only :port :engine))]
    conn_details))
 
 (def ^:private database-type->base-type