Skip to content

WPB-23631: Cache the public key bundle at startup instead of an effect - #5393

Open
blackheaven wants to merge 6 commits into
developfrom
gdifolco/WPB-23631-migrate-brig-galley-spar-effects-7
Open

WPB-23631: Cache the public key bundle at startup instead of an effect#5393
blackheaven wants to merge 6 commits into
developfrom
gdifolco/WPB-23631-migrate-brig-galley-spar-effects-7

Conversation

@blackheaven

Copy link
Copy Markdown
Contributor

https://wearezeta.atlassian.net/browse/WPB-23631

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

@blackheaven
blackheaven requested review from a team as code owners July 29, 2026 18:00
@blackheaven blackheaven added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Jul 29, 2026

@akshaymankar akshaymankar left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Its weird that this is an effect. We should read the file at startup and keep it cached instead of reading it all the time.

@blackheaven blackheaven changed the title WPB-23631: Move Brig.Effects.PublicKeyBundle in Wire.PublicKeyBundle WPB-23631: Cache the public key bundle at startup instead of an effect Aug 3, 2026
@blackheaven
blackheaven force-pushed the gdifolco/WPB-23631-migrate-brig-galley-spar-effects-7 branch from f7f9ea1 to a4c1acc Compare August 3, 2026 15:21
Move the PublicKeyBundle effect + interpreter to Wire.PublicKeyBundle
(single module). Rewire consumers (CanonicalInterpreter, Brig.API.Public,
Brig.User.Client); delete old module; register in cabal.
@blackheaven
blackheaven force-pushed the gdifolco/WPB-23631-migrate-brig-galley-spar-effects-7 branch from cdc3123 to a903dfe Compare August 3, 2026 19:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants