Skip to content

Commit

Permalink
Merge pull request #4782 from stacks-network/feat/bootup-message-sign…
Browse files Browse the repository at this point in the history
…er-local-node

warn for not using local stacks node with the signer
  • Loading branch information
ASuciuX authored May 22, 2024
2 parents 79d212d + 8330eb0 commit c06473b
Showing 1 changed file with 10 additions and 2 deletions.
12 changes: 10 additions & 2 deletions stacks-signer/src/v1/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -25,8 +25,8 @@ use std::sync::mpsc::{channel, Receiver, Sender};

use libsigner::v1::messages::SignerMessage;
use libsigner::SignerEventReceiver;
use slog::slog_info;
use stacks_common::info;
use slog::{slog_info, slog_warn};
use stacks_common::{info, warn};
use wsts::state_machine::OperationResult;

use crate::config::GlobalConfig;
Expand All @@ -53,6 +53,14 @@ pub struct SpawnedSigner {
impl From<GlobalConfig> for SpawnedSigner {
fn from(config: GlobalConfig) -> Self {
let endpoint = config.endpoint;
warn!(
"Reminder: The signer is primarily designed for use with a local or subnet network stacks node. \
It's important to exercise caution if you are communicating with an external node, \
as this could potentially expose sensitive data or functionalities to security risks \
if additional proper security checks are not integrated in place. \
For more information, check the documentation at \
https://docs.stacks.co/nakamoto-upgrade/signing-and-stacking/faq#what-should-the-networking-setup-for-my-signer-look-like."
);
info!("Starting signer with config: {}", config);
let (cmd_send, cmd_recv) = channel();
let (res_send, res_recv) = channel();
Expand Down

0 comments on commit c06473b

Please sign in to comment.