Skip to content

fix(tlock): guard Drand freshness timestamp overflow (#279) - #293

Merged
karagozemin merged 1 commit into
Sub-Rosa-Issue:mainfrom
CemAyyildiz:fix/issue-279-guard-drand-timestamp-overflow
Aug 31, 2026
Merged

fix(tlock): guard Drand freshness timestamp overflow (#279)#293
karagozemin merged 1 commit into
Sub-Rosa-Issue:mainfrom
CemAyyildiz:fix/issue-279-guard-drand-timestamp-overflow

Conversation

@CemAyyildiz

@CemAyyildiz CemAyyildiz commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add computePublishAtMs with Number.isSafeInteger guards for round/period math.
  • Classify overflow cases as unknown instead of producing unsafe timestamps.
  • Add Number.MAX_SAFE_INTEGER boundary tests.

Closes #279

Made with Cursor

@CemAyyildiz @cursoragent
fix(tlock): guard Drand freshness timestamp overflow (Sub-Rosa-Issue#279
6d98763 
)

Reject unsafe round/period timestamp math via computePublishAtMs and add
MAX_SAFE_INTEGER boundary tests while preserving valid classifications.

Co-authored-by: Cursor <cursoragent@cursor.com>
@drips-wave

drips-wave Bot commented Aug 31, 2026

Copy link
Copy Markdown

@CemAyyildiz Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@karagozemin
karagozemin merged commit b9ee40a into Sub-Rosa-Issue:main Aug 31, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Guard Drand freshness timestamp overflow

2 participants

@CemAyyildiz @karagozemin