GitBulletin

Repository: farcasterxyz/hub-monorepo

"DataDog Data Drama: New Prefix Protocol Saves the Day!"

2024-04-15 15:46:44 UTC

"Diagnostics Reports Revolutionized as Prefixes Added to fid and Peer ID, Ensuring Reliable Event Tracking"

Commit Details:

fix: update diagnostics reports to prefix tags with fid and peer_id (#1910) ## Motivation - For some reason DataDog ignores and omits the FID tag when it's just a "string-ified" number - When it's `fid:${fid.toString(10)}` it works - Add prefixes to fid and peer_id and type for consistency ## Change Summary ## Merge Checklist _Choose all relevant options below by adding an `x` now or at any time before submitting for review_ - [x] PR title adheres to the [conventional commits](https://www.conventionalcommits.org/en/v1.0.0/) standard - [x] PR has a [changeset](https://github.com/farcasterxyz/hub-monorepo/blob/main/CONTRIBUTING.md#35-adding-changesets) - [x] PR has been tagged with a change label(s) (i.e. documentation, feature, bugfix, or chore) - [ ] PR includes [documentation](https://github.com/farcasterxyz/hub-monorepo/blob/main/CONTRIBUTING.md#32-writing-docs) if necessary. - [x] All [commits have been signed](https://github.com/farcasterxyz/hub-monorepo/blob/main/CONTRIBUTING.md#22-signing-commits) ## Additional Context If this is a relatively large or complex change, provide more details here that will help reviewers <!-- start pr-codex --> --- ## PR-Codex overview This PR updates `diagnosticReportWorker.ts` in `@farcaster/hubble` to prefix tags with `fid` and `peer_id`. ### Detailed summary - Added `fid` and `peer_id` prefix to tags in `postDataDogEvent` function - Added `type:error` and `type:unavailable` tags for error and unavailable events respectively > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex -->

Search Similar