Skip to content

Commit 7233f47

Browse files
committed
Format code
1 parent be3676b commit 7233f47

9 files changed

Lines changed: 30 additions & 29 deletions

File tree

sentry-ktor-client/src/main/java/io/sentry/ktorClient/SentryKtorClientPlugin.kt

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -137,9 +137,7 @@ public val SentryKtorClientPlugin: ClientPlugin<SentryKtorClientPluginConfig> =
137137
request.headers.remove(BaggageHeader.BAGGAGE_HEADER)
138138
request.headers[it.name] = it.value
139139
}
140-
tracingHeaders.w3cTraceparentHeader?.let {
141-
request.headers[it.name] = it.value
142-
}
140+
tracingHeaders.w3cTraceparentHeader?.let { request.headers[it.name] = it.value }
143141
}
144142
}
145143
}

sentry-okhttp/src/main/java/io/sentry/okhttp/SentryOkHttpInterceptor.kt

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -116,9 +116,7 @@ public open class SentryOkHttpInterceptor(
116116
requestBuilder.removeHeader(BaggageHeader.BAGGAGE_HEADER)
117117
requestBuilder.addHeader(it.name, it.value)
118118
}
119-
tracingHeaders.w3cTraceparentHeader?.let {
120-
requestBuilder.addHeader(it.name, it.value)
121-
}
119+
tracingHeaders.w3cTraceparentHeader?.let { requestBuilder.addHeader(it.name, it.value) }
122120
}
123121
}
124122

sentry-okhttp/src/test/java/io/sentry/okhttp/SentryOkHttpInterceptorTest.kt

Lines changed: 11 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -648,10 +648,11 @@ class SentryOkHttpInterceptorTest {
648648

649649
@Test
650650
fun `adds W3C traceparent header when propagateTraceparent is enabled`() {
651-
val client = fixture.getSut(
652-
optionsConfiguration = Sentry.OptionsConfiguration { it.isPropagateTraceparent = true }
653-
)
654-
651+
val client =
652+
fixture.getSut(
653+
optionsConfiguration = Sentry.OptionsConfiguration { it.isPropagateTraceparent = true }
654+
)
655+
655656
fixture.server.enqueue(MockResponse().setResponseCode(200))
656657

657658
val request = getRequest("/test")
@@ -660,18 +661,19 @@ class SentryOkHttpInterceptorTest {
660661
val recordedRequest = fixture.server.takeRequest()
661662
assertNotNull(recordedRequest.getHeader("sentry-trace"))
662663
assertNotNull(recordedRequest.getHeader("traceparent"))
663-
664+
664665
val traceparent = recordedRequest.getHeader("traceparent")!!
665666
assertTrue(traceparent.startsWith("00-"))
666667
assertEquals(4, traceparent.split("-").size)
667668
}
668669

669670
@Test
670671
fun `does not add W3C traceparent header when propagateTraceparent is disabled`() {
671-
val client = fixture.getSut(
672-
optionsConfiguration = Sentry.OptionsConfiguration { it.isPropagateTraceparent = false }
673-
)
674-
672+
val client =
673+
fixture.getSut(
674+
optionsConfiguration = Sentry.OptionsConfiguration { it.isPropagateTraceparent = false }
675+
)
676+
675677
fixture.server.enqueue(MockResponse().setResponseCode(200))
676678

677679
val request = getRequest("/test")

sentry-openfeign/src/main/java/io/sentry/openfeign/SentryFeignClient.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -139,7 +139,8 @@ public Response execute(final @NotNull Request request, final @NotNull Request.O
139139
requestWrapper.header(baggageHeader.getName(), baggageHeader.getValue());
140140
}
141141

142-
final @Nullable W3CTraceparentHeader w3cTraceparentHeader = tracingHeaders.getW3cTraceparentHeader();
142+
final @Nullable W3CTraceparentHeader w3cTraceparentHeader =
143+
tracingHeaders.getW3cTraceparentHeader();
143144
if (w3cTraceparentHeader != null) {
144145
requestWrapper.header(w3cTraceparentHeader.getName(), w3cTraceparentHeader.getValue());
145146
}

sentry-spring-7/src/main/java/io/sentry/spring7/tracing/SentrySpanClientHttpRequestInterceptor.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,14 +6,14 @@
66

77
import com.jakewharton.nopen.annotation.Open;
88
import io.sentry.BaggageHeader;
9-
import io.sentry.W3CTraceparentHeader;
109
import io.sentry.Breadcrumb;
1110
import io.sentry.Hint;
1211
import io.sentry.IScopes;
1312
import io.sentry.ISpan;
1413
import io.sentry.SpanDataConvention;
1514
import io.sentry.SpanOptions;
1615
import io.sentry.SpanStatus;
16+
import io.sentry.W3CTraceparentHeader;
1717
import io.sentry.util.Objects;
1818
import io.sentry.util.SpanUtils;
1919
import io.sentry.util.TracingUtils;
@@ -115,7 +115,8 @@ private void maybeAddTracingHeaders(
115115
request.getHeaders().set(baggageHeader.getName(), baggageHeader.getValue());
116116
}
117117

118-
final @Nullable W3CTraceparentHeader w3cTraceparentHeader = tracingHeaders.getW3cTraceparentHeader();
118+
final @Nullable W3CTraceparentHeader w3cTraceparentHeader =
119+
tracingHeaders.getW3cTraceparentHeader();
119120
if (w3cTraceparentHeader != null) {
120121
request.getHeaders().add(w3cTraceparentHeader.getName(), w3cTraceparentHeader.getValue());
121122
}

sentry-spring-jakarta/src/main/java/io/sentry/spring/jakarta/tracing/SentrySpanClientHttpRequestInterceptor.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,14 +6,14 @@
66

77
import com.jakewharton.nopen.annotation.Open;
88
import io.sentry.BaggageHeader;
9-
import io.sentry.W3CTraceparentHeader;
109
import io.sentry.Breadcrumb;
1110
import io.sentry.Hint;
1211
import io.sentry.IScopes;
1312
import io.sentry.ISpan;
1413
import io.sentry.SpanDataConvention;
1514
import io.sentry.SpanOptions;
1615
import io.sentry.SpanStatus;
16+
import io.sentry.W3CTraceparentHeader;
1717
import io.sentry.util.Objects;
1818
import io.sentry.util.SpanUtils;
1919
import io.sentry.util.TracingUtils;
@@ -115,7 +115,8 @@ private void maybeAddTracingHeaders(
115115
request.getHeaders().set(baggageHeader.getName(), baggageHeader.getValue());
116116
}
117117

118-
final @Nullable W3CTraceparentHeader w3cTraceparentHeader = tracingHeaders.getW3cTraceparentHeader();
118+
final @Nullable W3CTraceparentHeader w3cTraceparentHeader =
119+
tracingHeaders.getW3cTraceparentHeader();
119120
if (w3cTraceparentHeader != null) {
120121
request.getHeaders().add(w3cTraceparentHeader.getName(), w3cTraceparentHeader.getValue());
121122
}

sentry-spring/src/main/java/io/sentry/spring/tracing/SentrySpanClientHttpRequestInterceptor.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,8 @@ private void maybeAddTracingHeaders(
107107
request.getHeaders().set(baggageHeader.getName(), baggageHeader.getValue());
108108
}
109109

110-
final @Nullable W3CTraceparentHeader w3cTraceparentHeader = tracingHeaders.getW3cTraceparentHeader();
110+
final @Nullable W3CTraceparentHeader w3cTraceparentHeader =
111+
tracingHeaders.getW3cTraceparentHeader();
111112
if (w3cTraceparentHeader != null) {
112113
request.getHeaders().add(w3cTraceparentHeader.getName(), w3cTraceparentHeader.getValue());
113114
}

sentry/src/main/java/io/sentry/W3CTraceparentHeader.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,5 +29,4 @@ public W3CTraceparentHeader(
2929
String sampledFlag = sampled != null && sampled ? "01" : "00";
3030
return String.format("%s-%s-%s", traceId, spanId, sampledFlag);
3131
}
32-
3332
}

sentry/src/test/java/io/sentry/util/TracingUtilsTest.kt

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -469,10 +469,10 @@ class TracingUtilsTest {
469469
assertNotNull(tracingHeaders.sentryTraceHeader)
470470
assertNotNull(tracingHeaders.w3cTraceparentHeader)
471471
assertEquals("traceparent", tracingHeaders.w3cTraceparentHeader!!.name)
472-
472+
473473
val headerValue = tracingHeaders.w3cTraceparentHeader!!.value
474474
assertTrue(headerValue.startsWith("00-"))
475-
475+
476476
val parts = headerValue.split("-")
477477
assertEquals(4, parts.size)
478478
assertEquals(fixture.span.spanContext.traceId.toString(), parts[1])
@@ -490,13 +490,13 @@ class TracingUtilsTest {
490490
assertNotNull(tracingHeaders)
491491
val w3cHeader = tracingHeaders.w3cTraceparentHeader!!
492492
assertEquals("traceparent", w3cHeader.name)
493-
493+
494494
val headerValue = w3cHeader.value
495495
assertTrue(headerValue.startsWith("00-"))
496-
496+
497497
val parts = headerValue.split("-")
498498
assertEquals(4, parts.size)
499-
499+
500500
val sentryTrace = fixture.span.toSentryTrace()
501501
val expectedFlag = if (sentryTrace.isSampled() == true) "01" else "00"
502502
assertEquals(expectedFlag, parts[3])
@@ -516,10 +516,10 @@ class TracingUtilsTest {
516516

517517
val w3cTrace = tracingHeaders.w3cTraceparentHeader!!
518518
assertEquals("traceparent", w3cTrace.name)
519-
519+
520520
val headerValue = w3cTrace.value
521521
assertTrue(headerValue.startsWith("00-"))
522-
522+
523523
val parts = headerValue.split("-")
524524
assertEquals(4, parts.size)
525525
}

0 commit comments

Comments
 (0)