Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Background refresh of actor/public key cache #1558

Merged
merged 1 commit into from
May 9, 2023

Conversation

bstasyszyn
Copy link
Contributor

The ActivityPub actor and public key caches are refreshed in the background so as not to affect the performance of a client call. If the refresh fails (due to the server being down) then the old value continues to be used.

closes #1551

The ActivityPub actor and public key caches are refreshed in the background so as not to affect the performance of a client call. If the refresh fails (due to the server being down) then the old value continues to be used.

closes trustbloc#1551

Signed-off-by: Bob Stasyszyn <Bob.Stasyszyn@securekey.com>
@cla-bot cla-bot bot added the cla-signed label May 8, 2023
@codecov
Copy link

codecov bot commented May 8, 2023

Codecov Report

Patch coverage: 95.02% and project coverage change: +0.07 🎉

Comparison is base (22f6010) 88.21% compared to head (2753831) 88.28%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1558      +/-   ##
==========================================
+ Coverage   88.21%   88.28%   +0.07%     
==========================================
  Files         226      227       +1     
  Lines       21186    21367     +181     
==========================================
+ Hits        18689    18864     +175     
- Misses       1602     1606       +4     
- Partials      895      897       +2     
Impacted Files Coverage Δ
internal/pkg/log/fields.go 96.85% <0.00%> (-1.87%) ⬇️
pkg/cache/cache.go 97.50% <97.50%> (ø)
cmd/orb-server/startcmd/start.go 67.52% <100.00%> (ø)
pkg/activitypub/client/client.go 86.11% <100.00%> (+0.55%) ⬆️

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@fqutishat fqutishat merged commit f77ef5f into trustbloc:main May 9, 2023
12 checks passed
@bstasyszyn bstasyszyn deleted the 1551 branch May 9, 2023 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Background refresh of actor/actor public key cache
2 participants