-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Added Defaults to ipfs daemon
#2652
Conversation
cmds.BoolOption(enableGCKwd, "Enable automatic periodic repo garbage collection"), | ||
cmds.BoolOption(adjustFDLimitKwd, "Check and raise file descriptor limits if needed"), | ||
cmds.BoolOption(initOptionKwd, "Initialize IPFS with default settings if not already initialized").Default(false), | ||
cmds.StringOption(routingOptionKwd, "Overrides the routing option (dht, supernode)").Default("supernode"), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the default is not supernode, lol
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is it? dht? :P
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is DHT.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What does 'supernode' mean here, then? I haven't run across this term before.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it was an old bit of code we wrote to be a 'centralized' routing system. We kindof abandoned it. The code might still work, but its not used.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, removed. We should remove that old code, then, too.
0820ce3
to
6e91bc5
Compare
Split from #2484. License: MIT Signed-off-by: Richard Littauer <richard.littauer@gmail.com>
6e91bc5
to
fa1c555
Compare
LGTM |
Split from #2484.
License: MIT
Signed-off-by: Richard Littauer richard.littauer@gmail.com