Skip to content
This repository has been archived by the owner on Apr 18, 2024. It is now read-only.

Ttfb should not be negative #56

Merged
merged 1 commit into from
Feb 28, 2023
Merged

Ttfb should not be negative #56

merged 1 commit into from
Feb 28, 2023

Conversation

aarshkshah1992
Copy link
Contributor

fixes #54 .

Copy link
Contributor

@guanzo guanzo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@lidel lidel merged commit f4deeda into main Feb 28, 2023
@lidel lidel deleted the fix/negative--ttfbms branch February 28, 2023 20:20
@guanzo
Copy link
Contributor

guanzo commented Mar 7, 2023

@aarshkshah1992 still getting negative values somehow

2023-03-07T00:44:44.633Z	17488e86-27d8-4e66-b0f5-5e67bb8ce449	INFO	Validation err: ttfbMs is negative {
  cacheHit: false,
  url: 'https://13.208.246.7/ipfs/QmfEWNbcZQfYDgePRvxDDGH9WoC4td3E1mgNLbRpDEoUuf?format=raw',
  startTime: '2023-03-07T00:44:38.210681964Z',
  numBytesSent: 181,
  requestDurationSec: 0.706223358,
  requestId: 'f478be4edbcb6d4103feec09f9cc8d38',
  httpStatusCode: 502,
  httpProtocol: 'HTTP/1.1',
  ttfbMs: -1678149878210,
  range: '',
  referrer: '',
  userAgent: 'bifrost-gateway/2023-03-01-adb95e1',
  nodeId: '8f3fffe6-5056-47d8-9c5c-6d44ed0c4ff6',
  ifNetworkError: '',
  nodeIpAddress: '13.208.246.7',
  host: '13.208.246.7',
  pathPrefix: 'ipfs',
  cid: 'QmfEWNbcZQfYDgePRvxDDGH9WoC4td3E1mgNLbRpDEoUuf',
  filePath: '',
  clientId: null,
  requestByteOffset: 0
}

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ensure TTFB metric isn't negative.
3 participants