Merge pull request #784 from tvandelooij/tvandelooij-patch-1

wangxiaoying · web-flow · commit 251b562d7e92 · 2025-04-03T11:26:54.000-07:00
URL decode database name for MSSQL
diff --git a/connectorx/src/sources/mssql/mod.rs b/connectorx/src/sources/mssql/mod.rs
@@ -59,7 +59,7 @@ pub fn mssql_config(url: &Url) -> Config {
     }
     config.port(url.port().unwrap_or(1433));
     // remove the leading "/"
-    config.database(&url.path()[1..]);
+    config.database(decode(&url.path()[1..])?.to_owned());
     // Using SQL Server authentication.
     #[allow(unused)]
     let params: HashMap<String, String> = url.query_pairs().into_owned().collect();

Original file line number	Diff line number	Diff line change
`@@ -59,7 +59,7 @@ pub fn mssql_config(url: &Url) -> Config {`
`59`	`59`	`}`
`60`	`60`	`config.port(url.port().unwrap_or(1433));`
`61`	`61`	`// remove the leading "/"`
`62`		`- config.database(&url.path()[1..]);`
	`62`	`+ config.database(decode(&url.path()[1..])?.to_owned());`
`63`	`63`	`// Using SQL Server authentication.`
`64`	`64`	`#[allow(unused)]`
`65`	`65`	`let params: HashMap<String, String> = url.query_pairs().into_owned().collect();`