RabbitMQ: Only set the AuthMechanisms when UseExternalAuthMechanism is set to true (#5416)

danielmarbach · web-flow · commit 4485ceace9f4 · 2026-04-13T12:17:44.000+02:00
Co-authored-by: Daniel Marbach &lt;danielmarbach@users.noreply.github.com&gt;
diff --git a/src/ServiceControl.Transports.RabbitMQ/RabbitMQTransportExtensions.cs b/src/ServiceControl.Transports.RabbitMQ/RabbitMQTransportExtensions.cs
@@ -38,9 +38,8 @@ public static void ApplySettingsFromConnectionString(this RabbitMQTransport tran
             transport.ValidateRemoteCertificate = !disableRemoteCertificateValidation;
         }
 
-        if (dictionary.TryGetValue("UseExternalAuthMechanism", out var useExternalAuthMechanismString))
+        if (dictionary.TryGetValue("UseExternalAuthMechanism", out var useExternalAuthMechanismString) && bool.TryParse(useExternalAuthMechanismString, out var useExternalAuthMechanism) && useExternalAuthMechanism)
         {
-            _ = bool.TryParse(useExternalAuthMechanismString, out var useExternalAuthMechanism);
             transport.AuthMechanisms = [new ExternalMechanismFactory()];
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -38,9 +38,8 @@ public static void ApplySettingsFromConnectionString(this RabbitMQTransport tran`
`38`	`38`	`transport.ValidateRemoteCertificate = !disableRemoteCertificateValidation;`
`39`	`39`	`}`
`40`	`40`
`41`		`- if (dictionary.TryGetValue("UseExternalAuthMechanism", out var useExternalAuthMechanismString))`
	`41`	`+ if (dictionary.TryGetValue("UseExternalAuthMechanism", out var useExternalAuthMechanismString) && bool.TryParse(useExternalAuthMechanismString, out var useExternalAuthMechanism) && useExternalAuthMechanism)`
`42`	`42`	`{`
`43`		`- _ = bool.TryParse(useExternalAuthMechanismString, out var useExternalAuthMechanism);`
`44`	`43`	`transport.AuthMechanisms = [new ExternalMechanismFactory()];`
`45`	`44`	`}`
`46`	`45`	`}`