Skip to content

Commit fe31bf6

Browse files
authored
Merge pull request #1204 from FinnWoelm/add-motherduck-connection-string-params
fix: add supported MotherDuck connection string parameters
2 parents c50bc82 + 001efd0 commit fe31bf6

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

duckdb_engine/config.py

+5-2
Original file line numberDiff line numberDiff line change
@@ -11,13 +11,16 @@
1111

1212
@lru_cache()
1313
def get_core_config() -> Set[str]:
14+
# List of connection string parameters that are supported by MotherDuck
15+
# See: https://motherduck.com/docs/key-tasks/authenticating-and-connecting-to-motherduck/authenticating-to-motherduck/
16+
motherduck_config_keys = {"motherduck_token", "attach_mode", "saas_mode"}
17+
1418
rows = (
1519
duckdb.connect(":memory:")
1620
.execute("SELECT name FROM duckdb_settings()")
1721
.fetchall()
1822
)
19-
# special case for motherduck here - they accept this config at extension load time
20-
return {name for (name,) in rows} | {"motherduck_token"}
23+
return {name for (name,) in rows} | motherduck_config_keys
2124

2225

2326
def apply_config(

0 commit comments

Comments
 (0)