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

Change Default Proxy Port #151

Open
phelixbtc opened this issue Aug 14, 2014 · 4 comments
Open

Change Default Proxy Port #151

phelixbtc opened this issue Aug 14, 2014 · 4 comments
Labels
prio:low

Comments

@phelixbtc
Copy link
Contributor

Most Tor users seem to use the Tor browser bundle these days. Tor browser bundle uses a different port, 9150 instead of 9050. Probably it would make things easier if we would switch the default to 9150. Or maybe put a note next to the option with both ports.
@JeremyRand
Copy link
Member

A note sounds like a good solution to me.

On August 14, 2014 1:56:59 PM CDT, phelixbtc notifications@github.com wrote:

Most Tor users seem to use the Tor browser bundle these days. Tor
browser bundle uses a different port, 9150 instead of 9050. Probably it
would make things easier if we would switch the default to 9150. Or
maybe put a note next to the option with both ports.

Reply to this email directly or view it on GitHub:
#151

Sent from my Android device with K-9 Mail. Please excuse my brevity.

@phelixbtc
Copy link
Contributor Author

@JeremyRand Why would you not switch the default port?

@phelixbtc
Copy link
Contributor Author

I still think we should switch the default port as most people seem to use the Tor browser bundle nowadays.

@JeremyRand
Copy link
Member

Sorry for dropping the ball on this issue. @phelixbtc I don't have a strong opinion on which port is the default, but I do think that if we change it, we should put a note next to the option with both ports.

domob1812 pushed a commit to domob1812/namecoin that referenced this issue Dec 16, 2014
@sipa
Squashed 'src/secp256k1/' changes from b0210a9..bccaf86
ecae2ac 
bccaf86 Merge pull request namecoin#150
2a53a47 Merge pull request namecoin#151
5f5a31f Merge pull request namecoin#149
3907277 Merge pull request namecoin#142
a3e0611 Enable tests in x86 travis builds
45da235 x86 builder
8bb0e93 Merge pull request namecoin#155
971fe81 build: fix openssl detection for cross builds
f22d73e Explicitly access %0..%2 as 64-bit so we use the right registers for x32 ABI
e66d4d6 Avoid the stack in assembly and use explicit registers
cf7b2b4 Fix ECDSA message hashes to 32 bytes
056ad31 Really compile with -O3 by default
74ad63a Merge pull request namecoin#146
9000458 Merge pull request namecoin#145
1f46b00 build: fix __builtin_expect detection for clang
aaba2e0 Merge pull request namecoin#136
8a0775c Merge pull request namecoin#144
ee1eaa7 Merge pull request namecoin#141
c88e2b8 Compile with -O3 by default
6558a26 Make the benchmarks print out stats
000bdf6 Rename bench_verify to bench_recovery
7c6fed2 Add a few more additional tests.
992e03b travis: add clang to the test matrix
b43b79a Merge pull request namecoin#143
e06a924 Include time.h header for time().
8d11164 Add some additional tests.
3545627 Merge pull request namecoin#118
6a9901e Merge pull request namecoin#137
376b28b Merge pull request namecoin#128
1728806 Merge pull request namecoin#138
a5759c5 Check return value of malloc
39bd94d Variable time normalize
ad86bdf Merge pull request namecoin#140
54b768c Another redundant secp256k1_fe_normalize
69dcaab Merge pull request #139
1c29f2e Remove redundant secp256k1_fe_normalize from secp256k1_gej_add_ge_var.
2b9388b Remove unused secp256k1_fe_inv_all
f461b76 Allocate precomputation arrays on the heap
b2c9681 Make {mul,sqr}_inner use the same argument order as {mul,sqr}
6793505 Convert YASM code into inline assembly
f048615 Rewrite field assembly to match the C version
3ce74b1 Tweak precomputed table size for G

git-subtree-dir: src/secp256k1
git-subtree-split: bccaf86caa9c44166e5a66600b742c516e03c3f0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
prio:low
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@JeremyRand @phelixbtc