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

Weaken language re (N-1) counter initialization approach #109

Closed
2 of 8 tasks
LiosK opened this issue May 28, 2022 · 0 comments · Fixed by #113
Closed
2 of 8 tasks

Weaken language re (N-1) counter initialization approach #109

LiosK opened this issue May 28, 2022 · 0 comments · Fixed by #113
Assignees

Comments

@LiosK
Copy link

LiosK commented May 28, 2022

Change Proposal Template

Source (Select one.)

  • IETF Published Draft
  • Work in Progress Draft

Change Reason (Select all that apply.)

  • Typos and grammatical issues
  • Bad Reference
  • IETF Verbiage modification (MAY, MUST, SHOULD, SHOULD NOT, etc)
  • New Text for additional context
  • Underlying XML Format Update
  • ASCII diagram updates (artwork, code samples, etc.)

Draft Number, Full Section, Name

6.2. Monotonicity and Counters - Counter Rollover Guards:

Current Text:

The technique from Fixed-Length Dedicated Counter Seeding which describes allocating a segment of the fixed-length counter as a rollover guard is also recommended and SHOULD be employed to help mitigate counter rollover issues. (snip)

Proposed Text:

The technique from Fixed-Length Dedicated Counter Seeding which describes allocating a segment of the fixed-length counter as a rollover guard is also helpful to mitigate counter rollover issues. (snip)


Other Supporting information below:

To align with the MAY wording used in the Fixed-Length Dedicated Counter Seeding: paragraph.

Implementations utilizing fixed-length counter method MAY also choose to randomly initialize a portion counter rather than the entire counter.

Further discussion: #60 (comment)

See also https://github.com/LiosK/infinite-uuid7-rs if you are interested in how often a counter rollover could occur. With a 42-bit counter, it almost never happens.

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

Successfully merging a pull request may close this issue.

2 participants