fix: Use correct types for event context data

getsentry · Sep 16, 2020 · 45f14a0 · 45f14a0
1 parent daae471
commit 45f14a0
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/packages/hub/src/scope.ts b/packages/hub/src/scope.ts
@@ -45,7 +45,7 @@ export class Scope implements ScopeInterface {
 
   /** Contexts */
   // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  protected _contexts: { [key: string]: any } = {};
+  protected _contexts: { [key: string]: Record<any, any> } = {};
 
   /** Fingerprint */
   protected _fingerprint?: string[];
@@ -185,9 +185,14 @@ export class Scope implements ScopeInterface {
   /**
    * @inheritDoc
    */
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  public setContext(key: string, context: { [key: string]: any } | null): this {
-    this._contexts = { ...this._contexts, [key]: context };
+  public setContext(key: string, context: { [key: string]: Record<any, any> } | null): this {
+    if (context === null) {
+      // eslint-disable-next-line @typescript-eslint/no-dynamic-delete
+      delete this._contexts[key];
+    } else {
+      this._contexts = { ...this._contexts, [key]: context };
+    }
+
     this._notifyScopeListeners();
     return this;
   }

diff --git a/packages/types/src/scope.ts b/packages/types/src/scope.ts
@@ -14,7 +14,7 @@ export interface ScopeContext {
   user: User;
   level: Severity;
   extra: { [key: string]: any };
-  contexts: { [key: string]: any };
+  contexts: { [key: string]: Record<any, any> };
   tags: { [key: string]: string };
   fingerprint: string[];
 }