fix: use url.host over url.hostname to include port numbers

Author: gingerbenw

packages/plugin-http-errors/http-errors.js

@@ -100,7 +100,7 @@ module.exports = (config = {}) => {
   const extractDomain = (url) => {
     try {
       const urlObj = new URL(url)
-      return urlObj.hostname
+      return urlObj.host
     } catch (e) {
       return 'unknown'
     }

packages/plugin-http-errors/test/http-errors.test.ts

@@ -648,7 +648,7 @@ describe('plugin-http-errors', () => {
 
       expect(notifyCallbacks.length).toBe(1)
       const event = notifyCallbacks[0]
-      expect(event.context).toBe('GET example.com')
+      expect(event.context).toBe('GET example.com:8080')
       expect(event.errors[0].errorMessage).toBe('404: https://example.com:8080/api/users')
     })
 

test/browser/features/http_errors.feature

@@ -29,3 +29,32 @@ Feature: HTTP Errors
     And the event "response.headers.content-length" equals "37"
     # response.body is not reported for fetch requests
     # response.bodyLength is not reported for fetch requests
+
+  @skip
+  Scenario: XML HTTP Request
+    When I navigate to the test URL "/http_errors/xhr/index.html"
+    And I wait to receive an error
+    Then the error is a valid browser payload for the error reporting API
+
+    And I define "expected.context" as "GET <browser.hostname>"
+    And I define "expected.exception.message" as "404: <browser.url>/reflect?status=404"
+    And I define "expected.request.url" as "<browser.url>/reflect?status=404"
+
+    And the exception "errorClass" equals "HTTPError"
+    And the error payload field "events.0.exceptions.0.message" equals the stored value "expected.exception.message"
+    And the event "severity" equals "error"
+    And the event "unhandled" is true
+    And the event "severityReason.type" equals "httpError"
+    And the error payload field "events.0.context" equals the stored value "expected.context"
+
+    And the error payload field "events.0.request.url" equals the stored value "expected.request.url"
+    And the event "request.httpMethod" equals "GET"
+    And the event "request.params.status" equals "404"
+    And the event "request.body" equals ""
+    And the event "request.bodyLength" equals 0
+
+    And the event "response.statusCode" equals 404
+    And the event "response.headers.content-length" equals "37"
+    And the event "response.body" equals ""
+    And the event "response.bodyLength" equals 37
+