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

Add bitcoinj Tor v3 support #5686

Merged
merged 2 commits into from
Sep 19, 2021

Conversation

ripcurlx
Copy link
Contributor

@ripcurlx ripcurlx commented Sep 9, 2021

This is atm only for those who want to help with testing our new bitcoinj fork on Regtest. If you want to test Tor v3 addresses against Mainnet only do it with a fresh data directory and DO NOT try to use it for real transactions as we haven't done intensive testing yet (only automated Regtest testing)

If you need a Bitcoin Core Node Tor v3 address for testing you can use mine (this is one of the federated Bitcoin core ndoes): catlnkpdm454ecngktyo4z22m5dlcvfvgzz4nt5l36eeczecrafslkqd.onion

e.g. use --btcNodes=catlnkpdm454ecngktyo4z22m5dlcvfvgzz4nt5l36eeczecrafslkqd.onion as startup argument.

Afterwards you should see following in your Network Info Tab

Bildschirmfoto 2021-09-09 um 11 40 16

…f578f427d78242bc2)

 Conflicts:
	build.gradle
	gradle/witness/gradle-witness.gradle
@ripcurlx ripcurlx marked this pull request as ready for review September 13, 2021 09:39
@ripcurlx ripcurlx added this to the v1.7.4 milestone Sep 13, 2021
@ripcurlx ripcurlx requested a review from sqrrm September 13, 2021 09:39
@ripcurlx
Copy link
Contributor Author

@jmacxx Could you also give this bitcoinj version a spin?

@Emzy
Copy link
Contributor

Emzy commented Sep 16, 2021

tested ACK
Build and used with 8 v3 onions in the file BtcNodes.java.

@ripcurlx
Copy link
Contributor Author

@sqrrm Could you please ACK and merge this PR as well so I can create the release branch for v1.7.4?

@ripcurlx
Copy link
Contributor Author

tested ACK
Build and used with 8 v3 onions in the file BtcNodes.java.

@Emzy Could you please provide a PR with the Tor v3 addresses for the v1.7.4 release? Thanks!

@Emzy
Copy link
Contributor

Emzy commented Sep 17, 2021

tested ACK
Build and used with 8 v3 onions in the file BtcNodes.java.

@Emzy Could you please provide a PR with the Tor v3 addresses for the v1.7.4 release? Thanks!

There are still some v3 onions from nodes missing. I'm on it.

Copy link
Member

@sqrrm sqrrm left a comment

Choose a reason for hiding this comment

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

ACK

@sqrrm sqrrm merged commit 4689ad5 into bisq-network:master Sep 19, 2021
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.

3 participants