Skip to content

Commit

Permalink
feat: enable recon by default (#270)
Browse files Browse the repository at this point in the history
Now that we have broken compatibility with Kubo APIs the only purpose of
ceramic-one is to run Recon. As such we should enable it by default as
the binary doesn't do anything meaningful with it disabled.
  • Loading branch information
nathanielc authored Feb 9, 2024
1 parent 6f4846a commit 573f9e6
Showing 1 changed file with 1 addition and 9 deletions.
10 changes: 1 addition & 9 deletions one/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -112,10 +112,6 @@ struct DaemonOpts {
#[arg(long, default_value_t = false, env = "CERAMIC_ONE_MDNS")]
mdns: bool,

/// When set Recon will be used to synchronized events with peers.
#[arg(long, default_value_t = false, env = "CERAMIC_ONE_RECON")]
recon: bool,

/// When set autonat will not be used to discover external address or allow other peers
/// to directly dial the local peer.
#[arg(long, default_value_t = false, env = "CERAMIC_ONE_DISABLE_AUTONAT")]
Expand Down Expand Up @@ -460,11 +456,7 @@ impl Daemon {
recon_metrics.clone(),
));

let recons = if opts.recon {
Some((recon_interest.client(), recon_model.client()))
} else {
None
};
let recons = Some((recon_interest.client(), recon_model.client()));
let ipfs_metrics =
ceramic_metrics::MetricsHandle::register(ceramic_kubo_rpc::IpfsMetrics::register);
let p2p_metrics = ceramic_metrics::MetricsHandle::register(ceramic_p2p::Metrics::register);
Expand Down

0 comments on commit 573f9e6

Please sign in to comment.