Skip to content

Commit

Permalink
chore: add missing response time (#465)
Browse files Browse the repository at this point in the history
  • Loading branch information
codecapitano authored Jan 24, 2024
1 parent 7444ea8 commit 00cb194
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 0 deletions.
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,8 @@

## Next

- Enhancement (`@grafana/faro-web-sdk`): add response time to performance timings (#465).

## 1.3.6

- Feature preview (`@grafana/faro-web-sdk`): instrument navigation and resource timings. As long as
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@ describe('performanceUtils', () => {
tlsNegotiationTime: '33',
redirectTime: '1',
requestTime: '109',
responseTime: '0',
fetchTime: '305',
serviceWorkerTime: '237',
decodedBodySize: '530675',
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,7 @@ export function calculateFaroResourceTiming(resourceEntryRaw: PerformanceResourc
tlsNegotiationTime: toFaroPerformanceTimingString(requestStart - secureConnectionStart),
redirectTime: toFaroPerformanceTimingString(redirectEnd - redirectStart),
requestTime: toFaroPerformanceTimingString(responseStart - requestStart),
responseTime: toFaroPerformanceTimingString(responseEnd - responseStart),
fetchTime: toFaroPerformanceTimingString(responseEnd - fetchStart),
serviceWorkerTime: toFaroPerformanceTimingString(fetchStart - workerStart),
decodedBodySize: toFaroPerformanceTimingString(decodedBodySize),
Expand Down
1 change: 1 addition & 0 deletions packages/web-sdk/src/instrumentations/performance/types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@ export type FaroResourceTiming = Readonly<{
redirectTime: string;
requestTime: string;
fetchTime: string;
responseTime: string;
serviceWorkerTime: string;
decodedBodySize: string;
encodedBodySize: string;
Expand Down

0 comments on commit 00cb194

Please sign in to comment.