-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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 hint to upgrade to tor v3 #5573
Add hint to upgrade to tor v3 #5573
Conversation
…ddress matches contract
whenever the client is in a state to do so without any problems.
After the date specified take offer requests by nodes with Tor v2 addresses will be rejected and their offers will be invisible to everyone else.
Local dev environment normally uses localhost:PORT addresses not connecting through Tor.
To test this on Regtest you have to modify the |
@pazza83 Does the timeline sounds reasonable to you? |
Suggestion:
I think it makes sense to leave out some of the details since they're already mentioned in the link. |
Based on feedback by @m52go
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.
Some comments but looks mostly good. I haven't tested and looks somewhat annoying to test on regtest, have you tested it?
desktop/src/main/java/bisq/desktop/main/support/dispute/agent/DisputeAgentView.java
Outdated
Show resolved
Hide resolved
desktop/src/main/java/bisq/desktop/main/portfolio/failedtrades/FailedTradesDataModel.java
Outdated
Show resolved
Hide resolved
desktop/src/main/java/bisq/desktop/main/support/dispute/DisputeView.java
Show resolved
Hide resolved
Yes, I did test it on Regtest. |
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.
utACK
The hint is great but after going to backup, it's gone and I have a hard time finding the advertised video on how to switch to v3. |
@Giszmo ah didn't think of that. Here are the links: |
I found the links by searching for this PR ;D Just wonder how non-developers are supposed to find them cause I took the non-developer route, first and googled for 10 minutes. The video did come up quickly but I had dismissed it, as it doesn't have "TORv3" in the title and was 45min long. |
"Tor v3" is in the title on both YouTube and BitcoinTV but I guess it doesn't matter if neither shows up. Maybe we should put the links and some copy on the backup screen that the primary call-to-action (green button) leads to in the next release? This might allow for reducing text in the modal and putting details in a less ephemeral place users can refer back to (i.e. the backup screen). |
True - I missed that people might not click on the links in the popups first before doing the backup. |
If the node is in a state where an upgrade to a Tor v3 address can be done it displays a informational prompt.
@m52go ☝️ This needs some native speaker love.
I'll check this state on application start and when the last open trade is completed. I wanted to do this for closed disputes as well, but the data structure in place would force me to scatter check codes on multiple places which would make it burdensome to remove after the transition is complete.
To have a graceful upgrade proces nodes with a Tor v2 address won't be able to take offers anymore starting with August 15, 2020 (2 months before the Tor v2 support is dropped). Also their offers won't be visible to anyone else at that point.
I also added some unrelated checks to this upgrade process, that you can only reopen trades and disputes if you have the same local onion address to prevent communication issues that would go by without notice.
This fixes #5558.