This is an automated email from the ASF dual-hosted git repository. elsloo pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-trafficcontrol.git
commit 6110f585386c0d677fddd31a71c7c378781e94b2 Author: Jesse Rivas <jesse_ri...@comcast.com> AuthorDate: Fri Jan 19 11:27:56 2018 -0700 initial commit --- .../cdn/traffic_control/traffic_router/core/http/RouterFilter.java | 4 ++-- .../cdn/traffic_control/traffic_router/core/router/TrafficRouter.java | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/traffic_router/core/src/main/java/com/comcast/cdn/traffic_control/traffic_router/core/http/RouterFilter.java b/traffic_router/core/src/main/java/com/comcast/cdn/traffic_control/traffic_router/core/http/RouterFilter.java index efdbad9..6702741 100644 --- a/traffic_router/core/src/main/java/com/comcast/cdn/traffic_control/traffic_router/core/http/RouterFilter.java +++ b/traffic_router/core/src/main/java/com/comcast/cdn/traffic_control/traffic_router/core/http/RouterFilter.java @@ -123,8 +123,8 @@ public class RouterFilter extends OncePerRequestFilter { } private void setMultiResponse(final HTTPRouteResult routeResult, final HttpServletRequest httpServletRequest, final HttpServletResponse response, final HTTPAccessRecord.Builder httpAccessRecordBuilder) throws IOException { - for (final DeliveryService deliveryService : routeResult.getDeliveryServices()) { - final Map<String, String> responseHeaders = deliveryService.getResponseHeaders(); + if (routeResult.getDeliveryService() != null) { + final Map<String, String> responseHeaders = routeResult.getDeliveryService().getResponseHeaders(); for (final String key : responseHeaders.keySet()) { // if two DSs append the same header, the last one wins; no way around it unless we enforce unique response headers between subordinate DSs diff --git a/traffic_router/core/src/main/java/com/comcast/cdn/traffic_control/traffic_router/core/router/TrafficRouter.java b/traffic_router/core/src/main/java/com/comcast/cdn/traffic_control/traffic_router/core/router/TrafficRouter.java index b63e075..acaa9d4 100644 --- a/traffic_router/core/src/main/java/com/comcast/cdn/traffic_control/traffic_router/core/router/TrafficRouter.java +++ b/traffic_router/core/src/main/java/com/comcast/cdn/traffic_control/traffic_router/core/router/TrafficRouter.java @@ -487,6 +487,7 @@ public class TrafficRouter { return null; } + routeResult.setDeliveryService(entryDeliveryService); for (final DeliveryService deliveryService : deliveryServices) { if (deliveryService.isRegionalGeoEnabled()) { LOGGER.error("Regional Geo Blocking is not supported with multi-route delivery services.. skipping " + entryDeliveryService.getId() + "/" + deliveryService.getId()); @@ -495,7 +496,6 @@ public class TrafficRouter { if (deliveryService.isAvailable()) { final List<Cache> caches = selectCaches(request, deliveryService, track); - routeResult.addDeliveryService(deliveryService); if (caches != null && !caches.isEmpty()) { final Cache cache = consistentHasher.selectHashable(caches, deliveryService.getDispersion(), request.getPath()); -- To stop receiving notification emails like this one, please contact els...@apache.org.