Need to account for cross-region-failback being null (not set in the application.yml)

aar0np · aar0np · commit 18a9a58173d2 · 2022-10-12T17:53:32.000-05:00
diff --git a/astra-spring-boot-autoconfigure/src/main/java/com/datastax/astra/boot/AstraSpringAutoConfiguration.java b/astra-spring-boot-autoconfigure/src/main/java/com/datastax/astra/boot/AstraSpringAutoConfiguration.java
@@ -109,7 +109,7 @@ public AstraClient astraClient() {
                 builder.withDatabaseRegion(api.getDatabaseRegion());
                 LOGGER.debug("+ Api /dbRegion detected {}", api.getDatabaseRegion());
             }
-            if (!api.getCrossRegionFailback()) {
+            if (api.getCrossRegionFailback() == null || !api.getCrossRegionFailback()) {
                 builder.disableCrossRegionFailOver();
                 LOGGER.debug("+ Cross Region Failback is disabled");
             }

Original file line number	Diff line number	Diff line change
`@@ -109,7 +109,7 @@ public AstraClient astraClient() {`
`109`	`109`	`builder.withDatabaseRegion(api.getDatabaseRegion());`
`110`	`110`	`LOGGER.debug("+ Api /dbRegion detected {}", api.getDatabaseRegion());`
`111`	`111`	`}`
`112`		`- if (!api.getCrossRegionFailback()) {`
	`112`	`+ if (api.getCrossRegionFailback() == null \|\| !api.getCrossRegionFailback()) {`
`113`	`113`	`builder.disableCrossRegionFailOver();`
`114`	`114`	`LOGGER.debug("+ Cross Region Failback is disabled");`
`115`	`115`	`}`