Fix order of http message handler to ensure unsuccessful requests can be traced

Author: C0nquistadore

src/Dibix.Http.Client/Client/DefaultHttpClientFactory.cs

@@ -341,14 +341,14 @@ public void ConfigureDefaults()
                 if (this.TraceProxy)
                     this.AddHttpMessageHandler<TraceProxyHttpMessageHandler>();
 
+                // Add this before each handler, that needs to access the response, before an exception is thrown. (i.E. TracingHttpMessageHandler)
+                if (this.EnsureSuccessStatusCode)
+                    this.AddHttpMessageHandler<EnsureSuccessStatusCodeHttpMessageHandler>();
+
                 // Run tracing after all other handlers, that potentially modified the request, to ensure the trace includes the actual request that is sent.
                 if (this.Tracer != null)
                     this.AddHttpMessageHandler(() => new TracingHttpMessageHandler(this.Tracer));
 
-                // Add this after each handler, that needs to access the response, before an exception is thrown. (i.E. TracingHttpMessageHandler)
-                if (this.EnsureSuccessStatusCode)
-                    this.AddHttpMessageHandler<EnsureSuccessStatusCodeHttpMessageHandler>();
-
                 // This should be as close to the primary handler as possible to avoid timeouts caused by other handlers.
                 if (this.WrapTimeoutsInException)
                     this.AddHttpMessageHandler<TimeoutHttpMessageHandler>();