chore: remove withAsync from ContextManager interface

open-telemetry · Apr 12, 2020 · 9cc34de · 9cc34de
1 parent f40f6f1
commit 9cc34de
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 19 deletions.
diff --git a/packages/opentelemetry-context-base/src/NoopContextManager.ts b/packages/opentelemetry-context-base/src/NoopContextManager.ts
@@ -26,10 +26,6 @@ export class NoopContextManager implements types.ContextManager {
     return fn();
   }
 
-  withAsync(context: Context, fn: Function) {
-    return fn();
-  }
-
   bind<T>(target: T, context?: Context): T {
     return target;
   }

diff --git a/packages/opentelemetry-context-base/src/types.ts b/packages/opentelemetry-context-base/src/types.ts
@@ -32,17 +32,6 @@ export interface ContextManager {
     fn: T
   ): ReturnType<T>;
 
-  /**
-   * Run the async fn callback with an active scope
-   *
-   * @param scope Context to be active during function invocation
-   * @param fn An async callback to be immediately run with an active context
-   */
-  withAsync<T extends Promise<any>, U extends (...args: unknown[]) => T>(
-    scope: Context,
-    fn: U
-  ): Promise<T>;
-
   /**
    * Bind an object as the current context (or a specific one)
    * @param target Any object to which a context need to be set

diff --git a/packages/opentelemetry-context-zone-peer-dep/src/ZoneContextManager.ts b/packages/opentelemetry-context-zone-peer-dep/src/ZoneContextManager.ts
@@ -248,8 +248,4 @@ export class ZoneContextManager implements ContextManager {
 
     return newZone.run(fn, context);
   }
-
-  withAsync(context: Context, fn: Function) {
-    return fn();
-  }
 }