search

adobe / helix-rum-js

Helix RUM Event Generator for JavaScript
Apache License 2.0
1 stars 8 forks source link

Use Performance API to get response status and sample accordingly #224

Closed phornig closed 1 week ago

phornig commented 1 week ago

Moved from fork to main repo

In cloud service depending on configuration we may not know the actual response status when injecting the RUM script, thus cannot determine if the 404 or standalone script should be included. This PR adds client-side code to determine the response status using the Performance API and trigger the 404 sampling when needed, effectively combining the 404 and standalone script into one.

The response code is read from the PerformanceNavigationTiming object. The responseStatus is supported by modern browsers except for Safari. As a fallback the ServerTiming object is inspected for a status property which is read from a server-timing header which needs to be injected by the server, e.g. in the CDN. This is also supported by Safari.

aem-code-sync[bot] commented 1 week ago

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed. In case there are problems, just click a checkbox below to rerun the respective action.

Commits * [8a3ec26](https://github.com/adobe/helix-rum-js/commit/8a3ec26db0fa920b0469d51bffa3d25e34b5c57d) :white_check_mark: (latest) * [d4f5994](https://github.com/adobe/helix-rum-js/commit/d4f5994d33c52498c5d5178712d6277cddf6fc0f) :white_check_mark: * [6daff20](https://github.com/adobe/helix-rum-js/commit/6daff2092621ffc7c7fc375b3962e571aa9782ec) :white_check_mark:
adobe-bot commented 1 week ago

:tada: This PR is included in version 2.9.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket: