Skip to content

fix: use configured localnet port in doctor command#47

Closed
ygd58 wants to merge 1 commit into
logos-co:masterfrom
ygd58:fix/doctor-hardcoded-port
Closed

fix: use configured localnet port in doctor command#47
ygd58 wants to merge 1 commit into
logos-co:masterfrom
ygd58:fix/doctor-hardcoded-port

Conversation

@ygd58
Copy link
Copy Markdown
Contributor

@ygd58 ygd58 commented Apr 9, 2026

Fixes #40.

Replace hardcoded port 3040 with project.config.localnet.port in build_doctor_report().

// Before
rows.push(check_port_warn(
    "sequencer port 3040",
    "127.0.0.1:3040",
    ...
));

// After
let localnet_port = project.config.localnet.port;
let localnet_addr = format!("127.0.0.1:{localnet_port}");
rows.push(check_port_warn(
    &format!("sequencer port {localnet_port}"),
    &localnet_addr,
    ...
));

Replace hardcoded port 3040 with project.config.localnet.port
in build_doctor_report().

Fixes logos-co#40
@ygd58 ygd58 requested a review from a team April 9, 2026 07:41
@weboko
Copy link
Copy Markdown
Collaborator

weboko commented Apr 10, 2026

duplicate of #41

@weboko weboko closed this Apr 10, 2026
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.

bug: doctor uses hardcoded port 3040 instead of configured localnet port

2 participants