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.

Sign up for GitHub

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

Jump to bottom

Using the --tor flag should default listen address to localhost only #39

Closed
davecgh opened this issue Nov 13, 2013 · 1 comment
Closed

Using the --tor flag should default listen address to localhost only #39

davecgh opened this issue Nov 13, 2013 · 1 comment
Labels
enhancement

Comments

@davecgh
Copy link
Member

davecgh commented Nov 13, 2013

Once #33 is implemented, the --tor flag should change the default listener from :8333 (all interfaces) to 127.0.0.1:8333 by default as it will ensure external peers can't connect via the public interface by default. The user will still be able to override this if they also want to provide public access via the --listen flag.

Also, tor does not work over IPv6, so the above default will stop listening on IPv6 by default when --tor is specified as well.
@davecgh
Copy link
Member Author

davecgh commented Nov 14, 2013

This no longer applies due to the way #33 was implemented and the forthcoming --externalip flag via #38. --proxy (with or without --tor) will disable listening by default. If the user still wants to listen, they can override it with --listen.

@davecgh davecgh closed this as completed Nov 14, 2013
davecgh pushed a commit to davecgh/btcd that referenced this issue Oct 11, 2017
@chappjc @alexlyp
Add Setticketsvotebits to dcrrpcclient(btcsuite#39)
6ef6822
jrick pushed a commit to jrick/btcd that referenced this issue Oct 12, 2017
@dajohi @alexlyp
Remove incorrect assignment (btcsuite#39)
9dd0f9f
jrick added a commit to jrick/btcd that referenced this issue Oct 12, 2017
@jrick
Revert "Remove incorrect assignment (btcsuite#39)"
a6bb98a 
This reverts commit 9dd0f9f.

This change alone would be a hard forking change to dcrd.  It must be
fixed at a later time, either as a soft fork, or on a voted-on hard
fork.
@githorray githorray mentioned this issue Jul 21, 2018
Open
kcalvinalvin added a commit to kcalvinalvin/btcd that referenced this issue Nov 29, 2024
@kcalvinalvin
Merge pull request btcsuite#39 from kcalvinalvin/add-reconsiderblock
17282cb 
blockchain: Add ReconsiderBlock
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@davecgh