Skip to content

docs: Clarify cache.ttl for 0, no expiry, and reads - #725

Merged
s1gr1d merged 2 commits into
mainfrom
sig/improve-ttl-2
Oct 19, 2026
Merged

s1gr1d merged 2 commits into
mainfrom
sig/improve-ttl-2

Conversation

@s1gr1d

@s1gr1d s1gr1d commented Oct 16, 2026 •

Copy link
Copy Markdown
Member

Description

Defines what cache.ttl means when its value is 0, when it is absent, and when it is set on a read span.

  • The value is the full TTL, not the time that remains
  • 0 means "expires immediately" for Sentry: Caches disagree on 0. HTTP max-age=0 (RFC 9111), DNS (RFC 1035), Django, Laravel, and Next.js treat it as "expire now". Memcached and Spring treat it as "never expires". The two SDKs that send cache.ttl (Next.js and Laravel) use "expire now".
  • An entry that never expires shouldn't send cache.ttl: Then no SDK has to send a number with two meanings. Next.js already does this. sentry-laravel currently sends null. Checked with @Litarnus - this is gonna be updated in Laravel.

Reference (Sentry Frontend): getsentry/sentry#127972

PR Checklist

  • I have run yarn test and verified that the tests pass.
  • I have run yarn generate 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 apply_scrubbing (i.e. manual or auto. Use never only for values that should never be scrubbed such as IDs)

If an attribute was deprecated:

@vercel

vercel Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
sentry-conventions Ready Ready Preview Oct 19, 2026 8:02am UTC

Request Review

@github-actions

github-actions Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 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 ✨

Attributes

  • Add object to the sentry.category values by pabloDeputter in #721
  • Add http.request.duration by loewenheim in #697

Other

  • (names) Add queue.publish/create/receive/process/settle to messaging span names by pabloDeputter in #705
  • (ops) Add object.copy op by pabloDeputter in #706
  • Add view.render op by alexander-alderman-webb in #699

Bug Fixes 🐛

  • (attributes) Apply consistent scrubbing across deprecations by mjq in #692

Documentation 📚

  • Clarify cache.ttl for 0, no expiry, and reads by s1gr1d in #725

Internal Changes 🔧

  • (repo) Update CODEOWNERS to include @giortzisg by Lms24 in #708
  • Deprecate gen_ai.responses in favor of gen_ai.chat by alexander-alderman-webb in #703
  • Add deprecated messaging ops set by the Python SDK by alexander-alderman-webb in #693
  • Add view.response.render op, deprecated in favor of view.render by alexander-alderman-webb in #700
  • Add event.django op, deprecated in favor of function by alexander-alderman-webb in #698

🤖 This preview updates automatically when you update the PR.

@Lms24 Lms24 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

thanks!

@s1gr1d
s1gr1d enabled auto-merge (squash) October 19, 2026 08:02
@s1gr1d
s1gr1d merged commit 617e756 into main Oct 19, 2026
17 checks passed
@s1gr1d
s1gr1d deleted the sig/improve-ttl-2 branch October 19, 2026 08:05

This branch was successfully deployed

1 active deployment
Preview — e5bf6c8b Deployed Oct 19, 2026 by vercel[bot]
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.

3 participants