Skip to content

Warn at startup when binding to a non-loopback address - #342

Open
SashaMIT wants to merge 1 commit into
axelarnetwork:mainfrom
SashaMIT:fix/warn-non-loopback-bind
Open

Warn at startup when binding to a non-loopback address#342
SashaMIT wants to merge 1 commit into
axelarnetwork:mainfrom
SashaMIT:fix/warn-non-loopback-bind

Conversation

@SashaMIT

@SashaMIT SashaMIT commented Aug 6, 2026

Copy link
Copy Markdown

Summary

  • The multisig gRPC service is plaintext and unauthenticated: any caller that can reach the port can request signatures for every key this daemon holds.
  • The default bind is loopback (127.0.0.1), which keeps that power on the local machine. But --address 0.0.0.0 (or any non-loopback bind) silently exposes a signing oracle to the network.
  • Log a prominent startup warning when the bind address is not loopback, so the exposure is a deliberate operator choice rather than a configuration slip.

This mirrors the client-side warning for non-loopback tofnd-host values proposed in axelarnetwork/axelar-core#2387; together they cover both ends of the channel.

Test plan

  • cargo check clean; cargo clippy introduces no new warnings.
  • Default 127.0.0.1 bind logs nothing new; --address 0.0.0.0 logs the warning at startup.

Made with Cursor

Made with Cursor

The multisig gRPC service is plaintext and unauthenticated: any caller
that can reach the port can request signatures for every key this
daemon holds. The default bind is loopback, which keeps that power on
the local machine, but --address 0.0.0.0 silently exposes a signing
oracle to the network. Log a prominent warning at startup when the bind
address is not loopback so the exposure is a deliberate operator choice
rather than a configuration slip.

Mirrors the client-side warning for non-loopback tofnd hosts in
axelarnetwork/axelar-core#2387.
@SashaMIT
SashaMIT requested a review from a team as a code owner August 6, 2026 20:05
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