Skip to content

Conversation

@qlrd
Copy link
Contributor

@qlrd qlrd commented Jun 23, 2025

What is the purpose of this pull request?

  • Bug fix
  • Documentation update
  • New feature
  • Test
  • Other:

Which crates are being modified?

  • floresta-chain
  • floresta-cli
  • floresta-common
  • floresta-compact-filters
  • floresta-electrum
  • floresta-watch-only
  • floresta-wire
  • floresta
  • florestad
  • Other:

Description

Notes to the reviewers

Previously, if we passed --rpc-address=127.0.0.1:0 it didn't log the port choosen by system. Now this log the correct port.

Author Checklist

  • I've followed the contribution guidelines
  • I've verified one of the following:
    • Ran just pcc (recommended but slower)
    • Ran just lint-features '-- -D warnings' && cargo test --release
    • Confirmed CI passed on my fork
  • I've linked any related issue(s) in the sections above

@Davidson-Souza Davidson-Souza added the code quality Generally improves code readability and maintainability label Jun 23, 2025
* This commit fixes the correct ip and port information returned from
TcpListener.
@qlrd qlrd changed the title Fix action ip:port address fore log RPC info Fix action ip:port address for log RPC info Jun 24, 2025
Copy link
Contributor

@JoseSK999 JoseSK999 left a comment

Choose a reason for hiding this comment

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

ACK 4a89412

Copy link
Member

@Davidson-Souza Davidson-Souza left a comment

Choose a reason for hiding this comment

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

ACK 4a89412

@Davidson-Souza Davidson-Souza merged commit 9503e63 into vinteumorg:master Jun 24, 2025
10 checks passed
@qlrd qlrd deleted the fix-log-rpc branch September 2, 2025 20:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

code quality Generally improves code readability and maintainability

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants