feat: Include updated_at timestamp in UserCacheService cache key #188
+37
−22
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.
Description
This PR enhances the
UserCacheServiceby incorporating the user'supdated_attimestamp into the cache key generation. This change enables automatic cache busting when user data is modified, improving cache consistency and data freshness.Changes Made
Core Changes
getCacheKey()to acceptUserobject and includeupdated_attimestamp in the keyget(),put(), andinvalidate()methods to acceptUserobjects instead of just user IDs$user->uuidconsistentlyController Updates
current()method to pass the full$userobject to cache service methodsDocumentation Updates
assignSingleRole()methodBenefits
✅ Automatic cache invalidation - When user data changes, the
updated_attimestamp changes, resulting in a new cache key✅ Improved data consistency - Ensures stale data is never served from cache
✅ Aligns with existing patterns - Matches the
generateETag()implementation which already usesupdated_at✅ Simplified cache management - Reduces reliance on explicit invalidation calls
✅ Better cache efficiency - Old cache entries expire naturally via TTL
Cache Key Format
Before:
user:current:{userId}:{companyId}After:
user:current:{userUuid}:{companyId}:{updatedAtTimestamp}Example:
user:current:abc-123-def:company-456:1707619200Testing Recommendations
Backward Compatibility
This change is backward compatible. Old cache keys will naturally expire based on the configured TTL (default 15 minutes). No migration or manual cache clearing is required.
Related
Implements the technical plan for improving user cache management by leveraging timestamp-based cache keys.