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

Tor 0.4.2.6 #4028

Merged
merged 2 commits into from
Mar 31, 2020
Merged

Tor 0.4.2.6 #4028

merged 2 commits into from
Mar 31, 2020

Conversation

freimair
Copy link
Member

@freimair freimair commented Mar 5, 2020

Part of bisq-network/projects#23

Contains:

  • Upgrade Tor to a version that defaults to HSv3.
  • Upgrade jtorctl to handle HSv3 properly.
  • Upgrade netlayer so it uses the new jtorctl and tor.

Note:

  • the previous versions of the libraries added timeouts checks to further dial in on the Tor issues we encounter. These changes are included here as well. However, that did not went so well for Windows and Bisq reverted the update.
  • for testing, I created a libs directory holding snapshots of the jtorctl and netlayer binaries. Once we are somewhat certain that they work, I am going to release them properly and remove the libs dir.

@devinbileck
Copy link
Member

devinbileck commented Mar 9, 2020

I tested this on a Windows 10 VM which has encountered Tor startup issues in the past and it worked successfully, for which it took 86 seconds to start Tor. Note that my testing only verified application startup and connecting to the Tor network.

@ripcurlx
Copy link
Contributor

@freimair There is no need to add the [Do not merge!] if you have it as a Draft PR, as no one can merge it in that case. As @devinbileck already started testing it on Windows, shall we test it on all supported OSes as well (I could take over macOS)?

@freimair
Copy link
Member Author

no need to add the [Do not merge!]

better safe than sorry (and you guys merged something recently that has not been fully tested...)

already started testing it on Windows,

I asked him to do the quick check to see if the issue he had last update is still there. And I will talk to him tomorrow about how to set up the testing framework.

Upgrade Tor to a version that defaults to HSv3.
Upgrade jtorctl to handle HSv3 properly.
Upgrade netlayer os it uses the new jtorctl and tor.
Now the tor binaries are properly released on their own. Thus, there
is no need for them to be in the Bisq repo. Added checksums as well.
@freimair freimair changed the title [Do not merge!] Tor 0.4.2.6 Tor 0.4.2.6 Mar 30, 2020
@freimair freimair marked this pull request as ready for review March 30, 2020 14:26
@freimair freimair requested a review from cbeams as a code owner March 30, 2020 14:26
@freimair
Copy link
Member Author

After testing

  • major platforms report green light (linux, macos, windows)
  • testing has been performed with v2 seednodes and v3 seednodes
  • ripcurlx even tested "upgrading" an existing client to v3 (by deleting the hs files)

all in all, no failures have been reported detailed test results

Thus, I released all binaries of the netlayer stack properly now and I ask you guys to do some smoke tests just to make sure I did not mess up the dependency references.

Copy link
Contributor

@ripcurlx ripcurlx left a comment

Choose a reason for hiding this comment

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

ACK

After running in-depth tests on macOS (see testpad) I ran with the current state a couple of smoke tests again (v2 address and afterwards upgrade to v3). So far looking good!

@ripcurlx ripcurlx merged commit c62c2d8 into bisq-network:master Mar 31, 2020
Migrate to Tor v3 onion service protocol automation moved this from In progress to Done Mar 31, 2020
@ripcurlx ripcurlx added this to the v1.3.0 milestone Apr 1, 2020
@freimair freimair deleted the tor_0.4.2.6 branch September 7, 2020 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants