Merge pull request #252 from pace/transport-dump-last

Use dump round tripper as last in the chain
pace · Apr 26, 2021 · 8b084f5 · 8b084f5
2 parents 0235e64 + ff9582a
commit 8b084f5
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/http/transport/default_transport.go b/http/transport/default_transport.go
@@ -10,10 +10,11 @@ func NewDefaultTransportChain() *RoundTripperChain {
 		&ExternalDependencyRoundTripper{},
 		NewDefaultRetryRoundTripper(),
 		&JaegerRoundTripper{},
-		NewDumpRoundTripperEnv(),
 		&LoggingRoundTripper{},
 		&LocaleRoundTripper{},
 		&RequestIDRoundTripper{},
+		// Ensure this is always last, in order to get the correct dump
+		NewDumpRoundTripperEnv(),
 	)
 }
 
@@ -25,9 +26,10 @@ func NewDefaultTransportChainWithExternalName(name string) *RoundTripperChain {
 		&ExternalDependencyRoundTripper{name: name},
 		NewDefaultRetryRoundTripper(),
 		&JaegerRoundTripper{},
-		NewDumpRoundTripperEnv(),
 		&LoggingRoundTripper{},
 		&LocaleRoundTripper{},
 		&RequestIDRoundTripper{},
+		// Ensure this is always last, in order to get the correct dump
+		NewDumpRoundTripperEnv(),
 	)
 }