chore: change the break to a return to make it easier to follow

kmruiz · kmruiz · commit cc927664a50f · 2025-08-06T13:20:20.000+02:00
diff --git a/src/common/connectionManager.ts b/src/common/connectionManager.ts
@@ -171,13 +171,12 @@ export class ConnectionManager extends EventEmitter<ConnectionManagerEvents> {
                 if (searchParams.get("authSource") === "$external") {
                     return "ldap";
                 }
-                break;
+                return "scram";
             // default should catch also null, but eslint complains
             // about it.
             case null:
             default:
                 return "scram";
         }
-        return "scram";
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -171,13 +171,12 @@ export class ConnectionManager extends EventEmitter<ConnectionManagerEvents> {`
`171`	`171`	`if (searchParams.get("authSource") === "$external") {`
`172`	`172`	`return "ldap";`
`173`	`173`	`}`
`174`		`- break;`
	`174`	`+ return "scram";`
`175`	`175`	`// default should catch also null, but eslint complains`
`176`	`176`	`// about it.`
`177`	`177`	`case null:`
`178`	`178`	`default:`
`179`	`179`	`return "scram";`
`180`	`180`	`}`
`181`		`- return "scram";`
`182`	`181`	`}`
`183`	`182`	`}`