Repository navigation
Conversation
|
📊 Test Coverage: ✅ Passed — 86% of changed lines covered, min 80% |
🛡️ Trivy security scan (CRITICAL,HIGH,MEDIUM,LOW)Go dependenciesNo findings at CRITICAL,HIGH,MEDIUM,LOW. Container imageNo findings at CRITICAL,HIGH,MEDIUM,LOW. |
Author
|
Self-review:
Retested on quick-start: a select served end to end, also with the registry stopped; lists deleted from Redis were reloaded within a minute. |
tdprajwal
force-pushed
the
feat/109-registry-active-cache
branch
from
November 11, 2026 05:25
85c9159 to
4a288d0
Compare
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
What
Adds a
registrycacheplugin that keeps every active registry record in Redis for 24h, reloaded daily, and makes signature lookup, the agreement check and provider routing read through it. Supersedes #123.Changes to existing records (a withdrawn key, an inactive participant, a changed baseUrl) can take up to the TTL to apply. New records apply immediately, and a refusal is rechecked with the registry.
Why
Most requests no longer need a registry call: two loads per day instead of one call per request.
Testing
Unit tests. On quick-start, an AgricultureFacility select was served end to end with the registry stopped.
Closes #109