Have one MaxRedirectException instance per Protocol instance

Author: Stephane Landelle

providers/netty3/src/main/java/org/asynchttpclient/providers/netty3/handler/Protocol.java

@@ -73,6 +73,7 @@ public abstract class Protocol {
     private final boolean hasResponseFilters;
     protected final boolean hasIOExceptionFilters;
     private final TimeConverter timeConverter;
+    private final MaxRedirectException maxRedirectException;
 
     public static final Set<Integer> REDIRECT_STATUSES = new HashSet<Integer>();
     static {
@@ -102,6 +103,8 @@ public Protocol(ChannelManager channelManager, AsyncHttpClientConfig config, Net
         hasResponseFilters = !config.getResponseFilters().isEmpty();
         hasIOExceptionFilters = !config.getIOExceptionFilters().isEmpty();
         timeConverter = config.getTimeConverter();
+        maxRedirectException = new MaxRedirectException("Maximum redirect reached: " + config.getMaxRedirects());
+        maxRedirectException.setStackTrace(new StackTraceElement[0]);
     }
 
     public abstract void handle(Channel channel, NettyResponseFuture<?> future, Object message) throws Exception;
@@ -130,7 +133,7 @@ protected boolean exitAfterHandlingRedirect(//
 
         if (followRedirect(config, request) && REDIRECT_STATUSES.contains(statusCode)) {
             if (future.incrementAndGetCurrentRedirectCount() >= config.getMaxRedirects()) {
-                throw new MaxRedirectException("Maximum redirect reached: " + config.getMaxRedirects());
+                throw maxRedirectException;
 
             } else {
                 // We must allow 401 handling again.

providers/netty4/src/main/java/org/asynchttpclient/providers/netty4/handler/Protocol.java

@@ -73,6 +73,7 @@ public abstract class Protocol {
     private final boolean hasResponseFilters;
     protected final boolean hasIOExceptionFilters;
     private final TimeConverter timeConverter;
+    private final MaxRedirectException maxRedirectException;
 
     public static final Set<Integer> REDIRECT_STATUSES = new HashSet<Integer>();
     static {
@@ -102,6 +103,8 @@ public Protocol(ChannelManager channelManager, AsyncHttpClientConfig config, Net
         hasResponseFilters = !config.getResponseFilters().isEmpty();
         hasIOExceptionFilters = !config.getIOExceptionFilters().isEmpty();
         timeConverter = config.getTimeConverter();
+        maxRedirectException = new MaxRedirectException("Maximum redirect reached: " + config.getMaxRedirects());
+        maxRedirectException.setStackTrace(new StackTraceElement[0]);
     }
 
     public abstract void handle(Channel channel, NettyResponseFuture<?> future, Object message) throws Exception;
@@ -130,7 +133,7 @@ protected boolean exitAfterHandlingRedirect(//
 
         if (followRedirect(config, request) && REDIRECT_STATUSES.contains(statusCode)) {
             if (future.incrementAndGetCurrentRedirectCount() >= config.getMaxRedirects()) {
-                throw new MaxRedirectException("Maximum redirect reached: " + config.getMaxRedirects());
+                throw maxRedirectException;
 
             } else {
                 // We must allow 401 handling again.