Merge pull request #52 from aar0np/main

clun · web-flow · commit 5570ddb4a180 · 2022-10-25T01:17:27.000+02:00
Need to account for cross-region-failback being null
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`	`}`