refactor: improve error message clarity for URL parsing failures

Copilot · dlevy-msft-sql · Copilot · commit c86e5034e5a1 · 2026-02-02T16:09:34.000Z
Co-authored-by: dlevy-msft-sql &lt;194277063+dlevy-msft-sql@users.noreply.github.com&gt;
diff --git a/msdsn/conn_str.go b/msdsn/conn_str.go
@@ -837,7 +837,7 @@ func splitConnectionStringURL(dsn string) (map[string]string, error) {
 	u, err := url.Parse(dsn)
 	if err != nil {
 		// Do not include the original error which may contain credentials
-		return res, fmt.Errorf("unable to parse connection string")
+		return res, fmt.Errorf("unable to parse connection string: invalid URL format")
 	}
 
 	if u.Scheme != "sqlserver" {

Original file line number	Diff line number	Diff line change
`@@ -837,7 +837,7 @@ func splitConnectionStringURL(dsn string) (map[string]string, error) {`
`837`	`837`	`u, err := url.Parse(dsn)`
`838`	`838`	`if err != nil {`
`839`	`839`	`// Do not include the original error which may contain credentials`
`840`		`- return res, fmt.Errorf("unable to parse connection string")`
	`840`	`+ return res, fmt.Errorf("unable to parse connection string: invalid URL format")`
`841`	`841`	`}`
`842`	`842`
`843`	`843`	`if u.Scheme != "sqlserver" {`