ref: Allow for backslashes in unix paths

Author: kamilogorek

packages/integrations/src/rewriteframes.ts

@@ -24,11 +24,18 @@ export class RewriteFrames implements Integration {
    * @inheritDoc
    */
   private readonly _iteratee: StackFrameIteratee = (frame: StackFrame) => {
-    // Check if the frame filename begins with `/` or a Windows-style prefix such as `C:\\`
-    if (frame.filename && /^(\/|[A-Z]:\\)/.test(frame.filename)) {
-      const filename = frame.filename
-        .replace(/^[A-Z]:/, '') // remove Windows-style prefix
-        .replace(/\\/g, '/'); // replace all `\\` instances with `/`
+    if (!frame.filename) {
+      return frame;
+    }
+    // Check if the frame filename begins with `/` or a Windows-style prefix such as `C:\`
+    const isWindowsFrame = /^[A-Z]:\\/.test(frame.filename);
+    const startsWithSlash = /^\//.test(frame.filename);
+    if (frame.filename && (isWindowsFrame || startsWithSlash)) {
+      const filename = isWindowsFrame
+        ? frame.filename
+            .replace(/^[A-Z]:/, '') // remove Windows-style prefix
+            .replace(/\\/g, '/') // replace all `\\` instances with `/`
+        : frame.filename;
       const base = this._root ? relative(this._root, filename) : basename(filename);
       frame.filename = `app:///${base}`;
     }

packages/integrations/test/rewriteframes.test.ts

@@ -16,7 +16,7 @@ describe('RewriteFrames', () => {
             filename: '/www/src/app/file1.js',
           },
           {
-            filename: '/www/src/app/file2.js',
+            filename: '/www/src/app/mo\\dule/file2.js',
           },
         ],
       },
@@ -31,7 +31,7 @@ describe('RewriteFrames', () => {
                   filename: '/www/src/app/file1.js',
                 },
                 {
-                  filename: '/www/src/app/file2.js',
+                  filename: '/www/src/app/mo\\dule/file2.js',
                 },
               ],
             },
@@ -99,13 +99,13 @@ describe('RewriteFrames', () => {
     it('transforms messageEvent frames', () => {
       const event = rewriteFrames.process(messageEvent);
       expect(event.stacktrace!.frames![0].filename).toEqual('app:///src/app/file1.js');
-      expect(event.stacktrace!.frames![1].filename).toEqual('app:///src/app/file2.js');
+      expect(event.stacktrace!.frames![1].filename).toEqual('app:///src/app/mo\\dule/file2.js');
     });
 
     it('transforms exceptionEvent frames', () => {
       const event = rewriteFrames.process(exceptionEvent);
       expect(event.exception!.values![0].stacktrace!.frames![0].filename).toEqual('app:///src/app/file1.js');
-      expect(event.exception!.values![0].stacktrace!.frames![1].filename).toEqual('app:///src/app/file2.js');
+      expect(event.exception!.values![0].stacktrace!.frames![1].filename).toEqual('app:///src/app/mo\\dule/file2.js');
     });
 
     it('trasforms windowsExceptionEvent frames', () => {
@@ -129,15 +129,15 @@ describe('RewriteFrames', () => {
       const event = rewriteFrames.process(messageEvent);
       expect(event.stacktrace!.frames![0].filename).toEqual('/www/src/app/file1.js');
       expect(event.stacktrace!.frames![0].function).toEqual('whoops');
-      expect(event.stacktrace!.frames![1].filename).toEqual('/www/src/app/file2.js');
+      expect(event.stacktrace!.frames![1].filename).toEqual('/www/src/app/mo\\dule/file2.js');
       expect(event.stacktrace!.frames![1].function).toEqual('whoops');
     });
 
     it('transforms exceptionEvent frames', async () => {
       const event = rewriteFrames.process(exceptionEvent);
       expect(event.exception!.values![0].stacktrace!.frames![0].filename).toEqual('/www/src/app/file1.js');
       expect(event.exception!.values![0].stacktrace!.frames![0].function).toEqual('whoops');
-      expect(event.exception!.values![0].stacktrace!.frames![1].filename).toEqual('/www/src/app/file2.js');
+      expect(event.exception!.values![0].stacktrace!.frames![1].filename).toEqual('/www/src/app/mo\\dule/file2.js');
       expect(event.exception!.values![0].stacktrace!.frames![1].function).toEqual('whoops');
     });
   });