chore: update localnet and wallet-service to their latest releases - #143
Merged
Merged
Conversation
canton-barebones 0.3.0 -> 0.5.0, which moves the Splice checkout from 0.6.11 to 0.7.5, so an existing .canton-localnet/ re-scaffolds and the stack needs a reset before the next run. wallet-service v0.1.4 -> v0.2.1, which reads holdings off one paged active-contracts snapshot instead of replaying the update stream. The old read fired thousands of requests per balance and tripped the validator ingress rate limit, which answered 503. The allowBuilds key carries the resolved tarball id, so it moves to the new commit sha with the ref.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
github-project-automation
Bot
moved this from Backlog
to Done
in Canton - dAppBooster (#390)
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
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.
Summary
No related issue. Both host-side dependencies had newer releases, and the wallet-service one fixes a balance read that could take the local validator down.
@bootnodedev/canton-barebones0.3.0 to 0.5.0, and@bootnodedev/canton-wallet-servicev0.1.4 to v0.2.1.wallet-service v0.2.x reads holdings off one paged active-contracts snapshot. The old read replayed the whole update stream in 100-record pages, fired thousands of requests per balance, and tripped the validator ingress rate limit, which answered 503 and put an nginx error page in the wallet.
barebones 0.5.0 moves its Splice checkout from 0.6.11 to 0.7.5.
Changes
allowBuildsnames the new wallet-service commit, so itspreparebuild still runs.Acceptance criteria
Test plan
Automated tests
pnpm lint,pnpm typecheck,pnpm build,pnpm test(615 tests across the three Node packages plus the root scripts),pnpm knip,pnpm docs:check,pnpm run check:anatomy. All green.Manual verification
pnpm installfrom the repo root. The wallet-servicepreparebuild runs and leavesdist/server.js.node scripts/localnet-config.mjs <dir>against an empty directory. It scaffolds a config, reports(missing), applies the two nginx flags, and pins Splice 0.7.5.pnpm run upon an existing.canton-localnet/. It re-scaffolds, reporting the Splice tag move.Breaking changes
The Splice tag moved, so an existing LocalNet has to be reset. That drops the ledger, which means the DAR has to be deployed and the bootstrap re-run before the dApp works again.
Checklist
Screenshots
None.