Skip to content

dgb(test): pin assembled CoinParams address-encoding + relay-policy vs oracle - #263

Merged
frstrtr merged 1 commit into
masterfrom
dgb/phase-b-address-relay-kat
Jun 20, 2026
Merged

dgb(test): pin assembled CoinParams address-encoding + relay-policy vs oracle#263
frstrtr merged 1 commit into
masterfrom
dgb/phase-b-address-relay-kat

Conversation

@frstrtr

@frstrtr frstrtr commented Jun 20, 2026

Copy link
Copy Markdown
Owner

Phase B pool/share conformance KAT — test-only, name-scoped to dgb_share_test, mirrors the #161/#249 lock pattern.

Pins the values make_coin_params() actually hands the pool, on both nets, against the DGB oracle frstrtr/p2pool-dgb-scrypt (switch-oracle Option B). Previously-unguarded:

  • mainnet address bytes 30 (D…) / 63 (S…), testnet 126 / 140
  • BECH32 HRPs dgb / dgbt
  • address_p2sh_version2 = 0 (disabled) — regression guard: the oracle has no secondary P2SH prefix; the prior =5 was an LTC-borrowed artifact. A silent revert fails loud here.
  • p2p_port 5024, worker_port 5025, dust_threshold 100000

3-bucket: address bytes + ports are bucket-1 isolation/identity primitives (keep per-coin, v36 AND v37); dust_threshold is local relay policy.

Scope: 1 file, +34 lines, dgb_share_test only (target already registered + allowlisted — no CMake/build.yml change). Cannot touch the #244/#248/#260 won-block / mempool-feed path. 9/9 suite green locally.

…s oracle

Phase B pool/share conformance KAT (test-only, dgb_share_test). Pins the
values make_coin_params() hands the pool on both nets against the DGB oracle
frstrtr/p2pool-dgb-scrypt: mainnet/testnet address bytes (30/63, 126/140),
BECH32 HRPs (dgb/dgbt), the DISABLED secondary P2SH prefix
(address_p2sh_version2=0; prior =5 was an LTC-borrowed artifact), p2p_port
5024, worker_port 5025, dust_threshold 100000.

3-bucket: address bytes + ports = bucket-1 isolation/identity (keep per-coin,
v36 and v37); dust_threshold = local relay policy. No prod change.
@frstrtr
frstrtr merged commit cca340d into master Jun 20, 2026
27 checks passed
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.

1 participant