ref: prefix all external and native modules with package name

Author: kamilogorek

packages/node/src/handlers.ts

@@ -3,11 +3,11 @@ import { getHubFromCarrier, Scope } from '@sentry/hub';
 import { SentryEvent, Severity } from '@sentry/types';
 import { forget } from '@sentry/utils/async';
 import { serialize } from '@sentry/utils/object';
-import { parse as parseCookie } from 'cookie';
+import * as cookie from 'cookie';
 import * as domain from 'domain';
 import * as http from 'http';
-import { hostname } from 'os';
-import { parse as parseUrl } from 'url';
+import * as os from 'os';
+import * as url from 'url';
 import { NodeClient } from './client';
 import { getCurrentHub } from './hub';
 
@@ -46,10 +46,10 @@ function extractRequestData(req: { [key: string]: any }): { [key: string]: strin
   // query string:
   //   node: req.url (raw)
   //   express, koa: req.query
-  const query = req.query || parseUrl(originalUrl || '', true).query;
+  const query = req.query || url.parse(originalUrl || '', true).query;
   // cookies:
   //   node, express, koa: req.headers.cookie
-  const cookies = parseCookie(headers.cookie || '');
+  const cookies = cookie.parse(headers.cookie || '');
   // body data:
   //   node, express, koa: req.body
   let data = req.body;
@@ -122,7 +122,7 @@ function parseRequest(
       ...event.request,
       ...extractRequestData(req),
     },
-    server_name: global.process.env.SENTRY_NAME || hostname(),
+    server_name: global.process.env.SENTRY_NAME || os.hostname(),
   };
 
   if (req.user) {

packages/node/src/integrations/console.ts

@@ -1,6 +1,6 @@
 import { Integration, Severity } from '@sentry/types';
 import { fill } from '@sentry/utils/object';
-import { format } from 'util';
+import * as util from 'util';
 import { getCurrentHub } from '../hub';
 
 /**
@@ -59,7 +59,7 @@ function consoleWrapper(originalModule: any): any {
           {
             category: 'console',
             level: sentryLevel,
-            message: format.apply(undefined, arguments),
+            message: util.format.apply(undefined, arguments),
           },
           {
             input: [...arguments],

packages/node/src/integrations/http.ts

@@ -1,16 +1,16 @@
 import { Integration } from '@sentry/types';
 import { fill } from '@sentry/utils/object';
-import { ClientRequest, ClientRequestArgs, IncomingMessage, ServerResponse } from 'http';
-import { inherits } from 'util';
+import * as http from 'http';
+import * as util from 'util';
 import { getCurrentHub } from '../hub';
 
-let lastResponse: ServerResponse | undefined;
+let lastResponse: http.ServerResponse | undefined;
 
 /**
  * Request interface which can carry around unified url
  * independently of used framework
  */
-interface SentryRequest extends IncomingMessage {
+interface SentryRequest extends http.IncomingMessage {
   __ravenBreadcrumbUrl?: string;
 }
 
@@ -20,7 +20,7 @@ interface SentryRequest extends IncomingMessage {
  * @param options url that should be returned or an object containing it's parts.
  * @returns constructed url
  */
-function createBreadcrumbUrl(options: string | ClientRequestArgs): string {
+function createBreadcrumbUrl(options: string | http.ClientRequestArgs): string {
   // We could just always reconstruct this from this.agent, this._headers, this.path, etc
   // but certain other http-instrumenting libraries (like nock, which we use for tests) fail to
   // maintain the guarantee that after calling origClientRequest, those fields will be populated
@@ -54,7 +54,7 @@ function loadWrapper(nativeModule: any): any {
       const origClientRequest = originalModule.ClientRequest;
       const clientRequest = function(
         this: SentryRequest,
-        options: ClientRequestArgs | string,
+        options: http.ClientRequestArgs | string,
         callback: () => void,
       ): any {
         // Note: this won't capture a breadcrumb if a response never comes
@@ -68,7 +68,7 @@ function loadWrapper(nativeModule: any): any {
         this.__ravenBreadcrumbUrl = createBreadcrumbUrl(options);
       };
 
-      inherits(clientRequest, origClientRequest);
+      util.inherits(clientRequest, origClientRequest);
 
       fill(clientRequest.prototype, 'emit', emitWrapper);
 
@@ -80,13 +80,13 @@ function loadWrapper(nativeModule: any): any {
       // it still points at orig ClientRequest after our monkeypatch; these reimpls
       // just get that reference updated to use our new ClientRequest
       fill(originalModule, 'request', function(): any {
-        return function(options: ClientRequestArgs, callback: () => void): any {
-          return new originalModule.ClientRequest(options, callback) as ClientRequest;
+        return function(options: http.ClientRequestArgs, callback: () => void): any {
+          return new originalModule.ClientRequest(options, callback) as http.ClientRequest;
         };
       });
 
       fill(originalModule, 'get', function(): any {
-        return function(options: ClientRequestArgs, callback: () => void): any {
+        return function(options: http.ClientRequestArgs, callback: () => void): any {
           const req = originalModule.request(options, callback);
           req.end();
           return req;
@@ -101,8 +101,8 @@ function loadWrapper(nativeModule: any): any {
 /**
  * Wrapper function for request's `emit` calls
  */
-function emitWrapper(origEmit: EventListener): (event: string, response: ServerResponse) => EventListener {
-  return function(this: SentryRequest, event: string, response: ServerResponse): any {
+function emitWrapper(origEmit: EventListener): (event: string, response: http.ServerResponse) => EventListener {
+  return function(this: SentryRequest, event: string, response: http.ServerResponse): any {
     // I'm not sure why but Node.js (at least in v8.X)
     // is emitting all events twice :|
     if (lastResponse === undefined || lastResponse !== response) {

packages/node/src/parsers.ts

@@ -1,7 +1,7 @@
 import { SentryEvent, SentryException, StackFrame } from '@sentry/types';
 import { readFileAsync } from '@sentry/utils/fs';
 import { snipLine } from '@sentry/utils/string';
-import { basename, dirname } from 'path';
+import * as path from 'path';
 import * as stacktrace from 'stack-trace';
 
 const LINES_OF_CONTEXT: number = 7;
@@ -30,7 +30,7 @@ function getTransaction(frame: StackFrame): string {
   return frame.module || frame.function ? `${frame.module || '?'} at ${frame.function || '?'}` : '<unknown>';
 }
 
-const mainModule: string = `${(require.main && require.main.filename && dirname(require.main.filename)) ||
+const mainModule: string = `${(require.main && require.main.filename && path.dirname(require.main.filename)) ||
   global.process.cwd()}/`;
 
 /** JSDoc */
@@ -40,8 +40,8 @@ function getModule(filename: string, base?: string): string {
   }
 
   // It's specifically a module
-  const file = basename(filename, '.js');
-  filename = dirname(filename); // tslint:disable-line:no-parameter-reassignment
+  const file = path.basename(filename, '.js');
+  filename = path.dirname(filename); // tslint:disable-line:no-parameter-reassignment
   let n = filename.lastIndexOf('/node_modules/');
   if (n > -1) {
     // /node_modules/ is 14 chars