Only log network request when getWrappedStream is called

embrace-io · Apr 19, 2024 · 6ba1f17 · 6ba1f17
1 parent 55dfc57
commit 6ba1f17
Showing 1 changed file with 4 additions and 8 deletions.
diff --git a/...ava/io/embrace/android/embracesdk/internal/network/http/EmbraceUrlConnectionDelegate.java b/...ava/io/embrace/android/embracesdk/internal/network/http/EmbraceUrlConnectionDelegate.java
@@ -347,8 +347,7 @@ public long getHeaderFieldLong(@NonNull String name, long defaultValue) {
  @Override
  @Nullable
  public Map<String, List<String>> getHeaderFields() {
- final long startTime = embrace.getInternalInterface().getSdkCurrentTime();
- cacheAndLogNetworkCall(startTime);
+ cacheNetworkCallData();
  return headerFields.get();
  }
 
@@ -359,14 +358,13 @@ private <R> R retrieveHeaderField(@Nullable String name,
  if (name == null) {
  return null;
  }
- long startTime = embrace.getInternalInterface().getSdkCurrentTime();
  if (shouldInterceptHeaderRetrieval(name)) {
  // Strip the content encoding and length headers, as we transparently ungzip the content
  return defaultValue;
  }
 
  R result = action.invoke();
- cacheAndLogNetworkCall(startTime);
+ cacheNetworkCallData();
 
  return result;
  }
@@ -461,18 +459,16 @@ public String getRequestProperty(@NonNull String key) {
  @Override
  public int getResponseCode() {
  identifyTraceId();
- long startTime = embrace.getInternalInterface().getSdkCurrentTime();
- cacheAndLogNetworkCall(startTime);
+ cacheNetworkCallData();
  return responseCode.get();
  }
 
  @Override
  @Nullable
  public String getResponseMessage() throws IOException {
  identifyTraceId();
- long startTime = embrace.getInternalInterface().getSdkCurrentTime();
  String responseMsg = this.connection.getResponseMessage();
- cacheAndLogNetworkCall(startTime);
+ cacheNetworkCallData();
  return responseMsg;
  }