fix: remove trailing slash from request url (#6542)

* fix: remove trailing slash from request url

* moves cast

* {}

src/tracing/errors.test.ts

@@ -10,6 +10,41 @@ Object.defineProperty(window.performance, 'getEntriesByType', {
 describe('beforeSend', () => {
   const ERROR = {} as ErrorEvent
 
+  describe('updateRequestUrl', () => {
+    test('should remove "/#" from the request URL', () => {
+      const event = {
+        request: {
+          url: 'https://app.uniswap.org/#/example',
+        },
+      } as ErrorEvent
+
+      beforeSend(event, {})
+      expect(event.request?.url).toBe('https://app.uniswap.org/example')
+    })
+
+    test('should remove trailing slash from the request URL', () => {
+      const event = {
+        request: {
+          url: 'https://app.uniswap.org/example/',
+        },
+      } as ErrorEvent
+
+      beforeSend(event, {})
+      expect(event.request?.url).toBe('https://app.uniswap.org/example')
+    })
+
+    test('should not modify the request URL if no changes are required', () => {
+      const event = {
+        request: {
+          url: 'https://app.uniswap.org/example',
+        },
+      } as ErrorEvent
+
+      beforeSend(event, {})
+      expect(event.request?.url).toBe('https://app.uniswap.org/example')
+    })
+  })
+
   describe('chunkResponseStatus', () => {
     afterEach(() => {
       jest.restoreAllMocks()

src/tracing/errors.ts

@@ -31,10 +31,14 @@ function isEthersRequestError(error: Error): error is Error & { requestBody: str
 // Since the interface currently uses HashRouter, URLs will have a # before the path.
 // This leads to issues when we send the URL into Sentry, as the path gets parsed as a "fragment".
 // Instead, this logic removes the # part of the URL.
+// It also removes trailing slashes, as they are not needed.
 // See https://romain-clement.net/articles/sentry-url-fragments/#url-fragments
 function updateRequestUrl(event: ErrorEvent) {
   if (event.request?.url) {
     event.request.url = event.request.url.replace('/#', '')
+    if (event.request.url.endsWith('/')) {
+      event.request.url = event.request.url.slice(0, -1)
+    }
   }
 }