Skip to content

chore: remove dead use-postage-snapshot flag and unused bootstrapping logic - #5318

Merged
martinconic merged 8 commits into
masterfrom
cleanup/remove-use-postage-snapshot
Feb 17, 2026
Merged

chore: remove dead use-postage-snapshot flag and unused bootstrapping logic#5318
martinconic merged 8 commits into
masterfrom
cleanup/remove-use-postage-snapshot

Conversation

@martinconic

@martinconic martinconic commented Jan 10, 2026

Copy link
Copy Markdown
Contributor

Checklist

  • I have read the coding guide.
  • My change requires a documentation update, and I have done it.
  • I have added tests to cover my changes.
  • I have filled out the description and linked the related issues.

Description

The use-postage-snapshot flag was intended to bootstrap the node by fetching a postage snapshot from a Swarm feed. However, the feed address in pkg/node/bootstrap.gowas hardcoded to a zero address, rendering this feature non-functional and dormant.
This implements #5317

Open API Spec Version Changes (if applicable)

Motivation and Context (Optional)

Related Issue (Optional)

Screenshots (if appropriate):

@acud acud added this to the v2.8.0 milestone Feb 4, 2026
@martinconic
martinconic marked this pull request as ready for review February 9, 2026 17:22
Comment thread cmd/bee/cmd/cmd.go Outdated
Comment thread cmd/bee/cmd/start.go Outdated
Comment thread pkg/node/bootstrap.go Outdated
Comment thread pkg/node/node.go Outdated
Comment thread pkg/node/node.go Outdated
Comment thread cmd/bee/cmd/cmd.go Outdated
Comment thread pkg/postage/interface.go Outdated
@martinconic
martinconic merged commit 1fd7740 into master Feb 17, 2026
34 of 38 checks passed
@martinconic
martinconic deleted the cleanup/remove-use-postage-snapshot branch February 17, 2026 14:13
flotob added a commit to solardev-xyz/freedom-browser that referenced this pull request Mar 28, 2026
Bee 2.7.1 removed this flag (ethersphere/bee#5318). It was never
functional — the snapshot feed address was hardcoded to zero.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants