plz

getsentry · Sep 30, 2020 · 56e2c62 · 56e2c62
1 parent 9fc9d6d
commit 56e2c62
Show file tree

Hide file tree

Showing 6 changed files with 15 additions and 9 deletions.
diff --git a/packages/apm/src/integrations/tracing.ts b/packages/apm/src/integrations/tracing.ts
@@ -853,7 +853,7 @@ export class Tracing implements Integration {
         Tracing._log(`[Tracing] Transaction: ${SpanStatus.Cancelled} since it maxed out maxTransactionDuration`);
         if (event.contexts && event.contexts.trace) {
           event.contexts.trace = {
-            ...event.contexts.trace,
+            ...(typeof event.contexts.trace === 'object' && event.contexts.trace),
             status: SpanStatus.DeadlineExceeded,
           };
           event.tags = {

diff --git a/packages/hub/src/scope.ts b/packages/hub/src/scope.ts
@@ -2,6 +2,8 @@
 import {
   Breadcrumb,
   CaptureContext,
+  Context,
+  Contexts,
   Event,
   EventHint,
   EventProcessor,
@@ -43,7 +45,7 @@ export class Scope implements ScopeInterface {
   protected _extra: Extras = {};
 
   /** Contexts */
-  protected _contexts: { [key: string]: Record<string, unknown> } = {};
+  protected _contexts: Contexts = {};
 
   /** Fingerprint */
   protected _fingerprint?: string[];
@@ -183,7 +185,7 @@ export class Scope implements ScopeInterface {
   /**
    * @inheritDoc
    */
-  public setContext(key: string, context: Record<string, unknown> | null): this {
+  public setContext(key: string, context: Context | null): this {
     if (context === null) {
       // eslint-disable-next-line @typescript-eslint/no-dynamic-delete
       delete this._contexts[key];

diff --git a/packages/types/src/context.ts b/packages/types/src/context.ts
@@ -0,0 +1,2 @@
+export type Context = unknown;
+export type Contexts = Record<string, Context>;
diff --git a/packages/types/src/event.ts b/packages/types/src/event.ts
@@ -1,5 +1,7 @@
 import { Breadcrumb } from './breadcrumb';
+import { Contexts } from './context';
 import { Exception } from './exception';
+import { Extras } from './extra';
 import { Request } from './request';
 import { CaptureContext } from './scope';
 import { SdkInfo } from './sdkinfo';
@@ -31,11 +33,9 @@ export interface Event {
   };
   stacktrace?: Stacktrace;
   breadcrumbs?: Breadcrumb[];
-  contexts?: {
-    [key: string]: Record<any, any>;
-  };
+  contexts?: Contexts;
   tags?: { [key: string]: string };
-  extra?: { [key: string]: any };
+  extra?: Extras;
   user?: User;
   type?: EventType;
   spans?: Span[];

diff --git a/packages/types/src/index.ts b/packages/types/src/index.ts
@@ -1,5 +1,6 @@
 export { Breadcrumb, BreadcrumbHint } from './breadcrumb';
 export { Client } from './client';
+export { Context, Contexts } from './context';
 export { Dsn, DsnComponents, DsnLike, DsnProtocol } from './dsn';
 export { ExtendedError } from './error';
 export { Event, EventHint } from './event';

diff --git a/packages/types/src/scope.ts b/packages/types/src/scope.ts
@@ -1,4 +1,5 @@
 import { Breadcrumb } from './breadcrumb';
+import { Context, Contexts } from './context';
 import { EventProcessor } from './eventprocessor';
 import { Extra, Extras } from './extra';
 import { Severity } from './severity';
@@ -14,7 +15,7 @@ export interface ScopeContext {
   user: User;
   level: Severity;
   extra: Extras;
-  contexts: { [key: string]: Record<string, unknown> };
+  contexts: Contexts;
   tags: { [key: string]: string };
   fingerprint: string[];
 }
@@ -82,7 +83,7 @@ export interface Scope {
    * @param name of the context
    * @param context Any kind of data. This data will be normailzed.
    */
-  setContext(name: string, context: Record<string, unknown> | null): this;
+  setContext(name: string, context: Context | null): this;
 
   /**
    * Sets the Span on the scope.
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		export type Context = unknown;
		export type Contexts = Record<string, Context>;