Add UUIDv47 implementation #28
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This commit adds support for UUIDv47, which allows storing time-ordered UUIDv7 in databases while emitting UUIDv4-looking facades at API boundaries. Key features: - SipHash-2-4 implementation for cryptographic hashing - mask/unmask functions for bidirectional UUID conversion - Time-ordered database storage with random-looking external IDs - Comprehensive test coverage (44 new tests) - Complete documentation with usage examples Files added: - lib/siphash.ex: SipHash-2-4 cryptographic hash function - lib/uuid_v47.ex: UUIDv47 mask/unmask functionality - test/siphash_test.exs: SipHash tests - test/uuid_v47_test.exs: UUIDv47 tests - guides/UUID_V47.md: Comprehensive documentation Benefits: - Excellent database index performance (time-ordered storage) - Privacy preservation (no timestamp leakage in APIs) - Deterministic and reversible with secret key - Cryptographically secure
Upgrades stream_data from 0.5.0 to 1.2.0 to resolve deprecation warnings about register_test/4 being replaced with register_test/6. This fixes 6 deprecation warnings in property-based tests.
Use Range.new/3 with explicit step parameter to avoid deprecation warnings when creating ranges that might go backwards (e.g., 1..0). This fixes the warning: Range.new/2 and first..last default to a step of -1 when last < first
Update delete function signature to match Ecto.Adapter.Schema behaviour which now requires 5 parameters including a 'returning' parameter. This fixes the warning: function delete/5 required by behaviour Ecto.Adapter.Schema is not implemented
Contributor
Author
|
@bitwalker any chance I can get a review or feedback on this? In ll fix the conflict but I am to make sure you are ok with this |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add UUIDv47 implementation with privacy-preserving facades
This commit adds support for UUIDv47, which allows storing time-ordered
UUIDv7 in databases while emitting UUIDv4-looking facades at API boundaries.
Key features:
Files added:
Benefits:
Also update some packages and fix some warnings