chore: bump go-libp2p-kad-dht to v0.42#11398
Merged
Merged
Conversation
guillaumemichel
changed the title
gammazero
reviewed
Jul 21, 2026
Bumps boxo, bringing the offline router retention change (records are served until their EOL, no receive-age cap by default; ipfs/boxo#1189) and a gateway 304 cache-freshness fix (ipfs/boxo#1188). The v0.43 "Unified IPNS record storage" highlight now spells out how retention works: with the offline router and the DHT sharing the /dht datastore prefix, online nodes drop value records 48h after storage (the DHT's value-store GC) while offline nodes keep them until EOL.
lidel
reviewed
Jul 21, 2026
lidel
left a comment
lidel
left a comment
Member
There was a problem hiding this comment.
Looks good to me. What I pushed and why:
- boxo side (ipfs/boxo#1189): restored the offline router to no age cap by default, it now serves a record until its EOL, so an offline node (no republisher) doesn't silently drop still-valid records after 48h. Online nodes keep that cap via the DHT's GC over the shared
/dhtstore. - here: bumped boxo to that commit and rewrote the "Unified IPNS record storage" changelog highlight to spell out online vs offline retention.
Needs ipfs/boxo#1189 to land first (go.mod points at its branch); the pin bumps to the merge commit once it merges.
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub. |
Pick up boxo main HEAD (4794174d), which includes the merged offline router value-store change (ipfs/boxo#1189). This brings the root module, the kubo-as-a-library example, and test/dependencies onto one boxo pin; they previously pointed at three different commits. - go-libp2p moves to ec408fcc as a transitive floor required by boxo - changelog: note the one-time full-datastore key scan on first start after upgrading, and update the pinned boxo and go-libp2p hashes
Spell out that the repo has three go.mod files that must stay on the same dependency versions, and that a bare `go mod tidy` only tidies one module and lets pins drift between them.
lidel
approved these changes
Jul 21, 2026
lidel
left a comment
lidel
left a comment
Member
There was a problem hiding this comment.
LGTM. Pushed two commits with sync and automation of dep bumps, CI is green.
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
3 participants
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.
Updates Kubo to the new unified IPNS/DHT value-record storage:
Depends on ipfs/boxo#1189.