Skip to content

Make transaction prioritization optional#257

Merged
Fi3 merged 1 commit into
dmnd-pool:masterfrom
lorbax:make-rpc-optional
May 6, 2026
Merged

Make transaction prioritization optional#257
Fi3 merged 1 commit into
dmnd-pool:masterfrom
lorbax:make-rpc-optional

Conversation

@lorbax
Copy link
Copy Markdown
Collaborator

@lorbax lorbax commented May 6, 2026

Enable transaction prioritization only when RPC_URL, RPC_USER, RPC_PWD, and RPC_FEE_DELTA are all configured.

When no RPC settings are present, log that prioritizing transactions is not enabled; when only some settings are present, log the missing-variable error. The API server can now start without RPC settings, and /api/tx returns service unavailable when prioritization is disabled.

Remove the implicit default fee delta fallback so RPC_FEE_DELTA is required and passed explicitly to BitcoindRpc.

Enable transaction prioritization only when RPC_URL, RPC_USER, RPC_PWD, and RPC_FEE_DELTA are all configured.

When no RPC settings are present, log that prioritizing transactions is not enabled; when only some settings are present, log the missing-variable error. The API server can now start without RPC settings, and /api/tx returns service unavailable when prioritization is disabled.

Remove the implicit default fee delta fallback so RPC_FEE_DELTA is required and passed explicitly to BitcoindRpc.
@Fi3 Fi3 merged commit 8626c1f into dmnd-pool:master May 6, 2026
4 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.

2 participants