Skip to content

Comments

feat: add sentry.log.sequence attribute to the spec#262

Draft
logaretm wants to merge 1 commit intomainfrom
awad/add-log-sequence-attr-def
Draft

feat: add sentry.log.sequence attribute to the spec#262
logaretm wants to merge 1 commit intomainfrom
awad/add-log-sequence-attr-def

Conversation

@logaretm
Copy link
Member

@logaretm logaretm commented Feb 20, 2026

This pull request introduces a new log attribute, sentry.log.sequence, to both the JavaScript and Python Sentry conventions libraries. It’s a monotonically incrementing integer used by the SDK to ensure correct log ordering when timestamps are identical.

The attribute addresses an issue that came up in Cloudflare’s runtime which freezes all timers and clock functions, causing all timestamp calls to return the same value. This can lead to logs being displayed out of order if they don’t get sent in one request. Instead, they’ll appear in the order of arrival in ingest, which is subject to request latencies and out-of-order request completion.

This is a proposal as we have yet to see what product thinks about this.

Related SDK PRs:

PR Checklist

  • I have run yarn test and verified that the tests pass.
  • I have run yarn generate && yarn format to generate and format code and docs.

If an attribute was added:

  • The attribute is in a namespace (e.g. nextjs.function_id, not function_id)
  • I have used the correct value for pii (i.e. maybe or true. Use false only for values that should never be scrubbed such as IDs)

If an attribute was deprecated:

@github-actions
Copy link

github-actions bot commented Feb 20, 2026

Semver Impact of This PR

🟡 Minor (new features)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


New Features ✨

  • Add sentry.log.sequence attribute to the spec by logaretm in #262

Bug Fixes 🐛

  • (gen_ai) Input and output token description by obostjancic in #261

Documentation 📚

  • Update README with up-to-date links by ericapisani in #258

Internal Changes 🔧

  • (deps) Bump svelte from 5.48.1 to 5.51.5 by dependabot in #260
  • (deps-dev) Bump tar from 7.5.7 to 7.5.8 by dependabot in #259

🤖 This preview updates automatically when you update the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant