Skip to content

feat: regtest support#135

Open
S1nus wants to merge 4 commits into
zcash:mainfrom
S1nus:c/regtest
Open

feat: regtest support#135
S1nus wants to merge 4 commits into
zcash:mainfrom
S1nus:c/regtest

Conversation

@S1nus

@S1nus S1nus commented Dec 19, 2025

Copy link
Copy Markdown

Adds improved support for Regtest. Currently hardcodes upgrade heights to 1, but could be updated to allow customizable upgrade heights.

Comment thread src/commands/inspect/block.rs
@S1nus
S1nus requested a review from str4d December 21, 2025 19:29
Comment thread src/remote.rs
Comment on lines +86 to +98
if params.is_regtest() {
match self {
Servers::Custom(servers) => return Ok(servers.first().expect("not empty")),
Servers::Hosted(op) => {
return Err(anyhow!(
"Regtest network requires a custom server.\n\
Use: --server localhost:9067 (or your lightwalletd address)\n\
Hosted servers ({:?}) only support mainnet and testnet.",
op
))
}
}
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

would be nice to default to localhost and/or allow config from file or env

@nuttycom

Copy link
Copy Markdown
Collaborator

Looks like this ended up highly conflicted with the SOCKS proxy PR that was just merged.

@S1nus

S1nus commented Feb 2, 2026

Copy link
Copy Markdown
Author

Looks like this ended up highly conflicted with the SOCKS proxy PR that was just merged.

thanks for reply. will look into this!

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.

4 participants