Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

daemon: add '--no-sync' arg to optionally disable blockchain sync #5195

Merged
merged 1 commit into from
Mar 19, 2019

Conversation

xiphon
Copy link
Contributor

@xiphon xiphon commented Feb 25, 2019

Allows to run a daemon that will connect to other peers, but won't download the blockchain.

Could be used in Monero GUI to find public RPC nodes by exploring P2P peers advertising such a service.

If --bootstrap-daemon-address auto gets implemented, the local daemon will automatically discover public nodes and provide transparent RPC proxy to the wallets.
Both CLI and GUI wallets will use automatically discovered public nodes with zero to no additional overhead on the wallet side.
Also --bootstrap-daemon-address auto --no-sync will be useful to users who don't want to store the blockchain locally.

As well it seems to be useful in conjunction with forthcoming share-rpc feature.

@Gingeropolous
Copy link
Collaborator

Just as an aside, I think this feature could also be used for a "listen-only" mode - i.e., you could imagine a phone wallet / node that has some pruned version of the blockchain (well, enough of it to craft a transaction with enough random inputs). This wallet / node needs to reduce resources, so it could just listen to the network on demand (like, when your standing next to your friend and say "send me some monero") and just scan the txpool. If the daemon sees the transaction, it downloads the block (or just the transaction).

This flag is cool because I think it begins to open the idea that you can use the monero daemon to interface with the monero network without requiring a copy of the blockchain.

@moneromooo-monero
Copy link
Collaborator

I would not say this flag is cool. It's more... pragmatic. Ideally it would not exist, because it means your node is useless to the network. It's like adding a broken ring to a chainmain. If you add enough, your chainmail becomes shit and useless.

{
context.m_state = cryptonote_connection_context::state_normal;
return true;
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

I'm not 100% sure this will be enough, but I suppose we can fix it if/when it pops up.

Copy link
Contributor

@fluffypony fluffypony left a comment

Choose a reason for hiding this comment

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

Reviewed

@fluffypony fluffypony merged commit a54e81e into monero-project:master Mar 19, 2019
fluffypony added a commit that referenced this pull request Mar 19, 2019
a54e81e daemon: add '--no-sync' arg to optionally disable blockchain sync (xiphon)
@xiphon xiphon deleted the no-sync branch July 14, 2019 11:23
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.

4 participants