Synamedia minor changes to regional geoblocking (#8126)

Minor changes to regional geoblocking:
* updated a debug message to be a bit clearerr
* return early from createRedirectURIString() if cache is null

Co-authored-by: Brett Granger <bgranger@synamedia.com>

Author: bdgranger

traffic_router/core/src/main/java/org/apache/traffic_control/traffic_router/core/loc/RegionalGeo.java

@@ -433,7 +433,7 @@ public static void enforce(final TrafficRouter trafficRouter, final Request requ
             if(!"Denied".equals(redirectURIString)){
                 routeResult.addUrl(new URL(redirectURIString));
             }else{
-                LOGGER.warn("RegionalGeo: this needs a better error message, createRedirectURIString returned denied");
+                LOGGER.debug("RegionalGeo: no availabe cache for relative path, 503 will be sent");
             }
         }
     }
@@ -468,6 +468,14 @@ private static void updateTrack(final Track track, final RegionalGeoResult regio
     private static String createRedirectURIString(final HTTPRequest request, final DeliveryService deliveryService, 
         final Cache cache, final RegionalGeoResult regionalGeoResult) {
 
+        if (regionalGeoResult.getType() == ALTERNATE_WITHOUT_CACHE) {
+            return regionalGeoResult.getUrl();
+        }
+
+        if (cache == null) {
+            return "Denied";
+        }
+
         if (regionalGeoResult.getType() == ALLOWED) {
             return deliveryService.createURIString(request, cache);
         }
@@ -476,10 +484,6 @@ private static String createRedirectURIString(final HTTPRequest request, final D
             return deliveryService.createURIString(request, regionalGeoResult.getUrl(), cache);
         }
 
-        if (regionalGeoResult.getType() == ALTERNATE_WITHOUT_CACHE) {
-            return regionalGeoResult.getUrl();
-        }
-
         return "Denied"; // DENIED
     }