HttpURLConnection response code edge case (#4062)
This commit is contained in:
parent
400d99467a
commit
2c044253e8
|
@ -124,7 +124,12 @@ public class HttpUrlConnectionInstrumentation implements TypeInstrumentation {
|
||||||
// to be consistent with the telemetry for other http clients.
|
// to be consistent with the telemetry for other http clients.
|
||||||
instrumenter().end(httpUrlState.context, connection, responseCode, null);
|
instrumenter().end(httpUrlState.context, connection, responseCode, null);
|
||||||
} else {
|
} else {
|
||||||
instrumenter().end(httpUrlState.context, connection, null, throwable);
|
instrumenter()
|
||||||
|
.end(
|
||||||
|
httpUrlState.context,
|
||||||
|
connection,
|
||||||
|
responseCode > 0 ? responseCode : null,
|
||||||
|
throwable);
|
||||||
}
|
}
|
||||||
httpUrlState.finished = true;
|
httpUrlState.finished = true;
|
||||||
} else if (methodName.equals("getInputStream") && responseCode > 0) {
|
} else if (methodName.equals("getInputStream") && responseCode > 0) {
|
||||||
|
|
Loading…
Reference in New Issue