Skip to content

Commit 07c1360

Browse files
authored
ref(otel): Ensure we never swallow args for ContextManager (#12798)
This is more a safety thing that we do not swallow this accidentally in the future, that I noticed while looking through stuff.
1 parent aefbfaa commit 07c1360

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

packages/opentelemetry/src/contextManager.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,8 +31,8 @@ export function wrapContextManagerClass<ContextManagerInstance extends ContextMa
3131

3232
// @ts-expect-error TS does not like this, but we know this is fine
3333
class SentryContextManager extends ContextManagerClass {
34-
public constructor() {
35-
super();
34+
public constructor(...args: unknown[]) {
35+
super(...args);
3636
setIsSetup('SentryContextManager');
3737
}
3838
/**

packages/opentelemetry/src/propagator.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -155,7 +155,7 @@ export class SentryPropagator extends W3CBaggagePropagator {
155155

156156
const propagationContext = propagationContextFromHeaders(sentryTrace, baggage);
157157

158-
// Add remote parent span context,
158+
// Add remote parent span context
159159
const ctxWithSpanContext = getContextWithRemoteActiveSpan(context, { sentryTrace, baggage });
160160

161161
// Also update the scope on the context (to be sure this is picked up everywhere)

0 commit comments

Comments
 (0)