doc: Update tor.md for notes on how to preserve v2 urls (with a not recommended note)#20957
doc: Update tor.md for notes on how to preserve v2 urls (with a not recommended note)#20957nolim1t wants to merge 1 commit intobitcoin:masterfrom
Conversation
|
Thanks @nolim1t. Unless this change is backported to 0.21.1, by the time 22.0 (the next release) is out tor v2 will be EOL, so maybe the existing release notes are enough. It may be good to squash your commits down to one when you are finished. |
25d798f to
7e08a1b
Compare
Squashed into one. Yep I recognize that it may be deprecated too, hence not recommended mention |
|
We're on the trailing edge of TorV2 support here, and this is a really hacky workaround. I too think that the mention in the release notes is enough. No strong objection though. |
jarolrod
left a comment
There was a problem hiding this comment.
Concept 0 on this, I think there's no need to mention this as it is will be mandatory to migrate to torv3 since support for v2 addresses will be disabled on the network, see Tor Deprecation Timeline
|
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers. ConflictsReviewers, this pull request conflicts with the following ones:
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first. |
|
🐙 This pull request conflicts with the target branch and needs rebase. Want to unsubscribe from rebase notifications on this pull request? Just convert this pull request to a "draft". |
|
I'm going to close this, sorry. Everyone is at most ~0 on this, which makes it unlikely that this will be merged. It doesn't seem worth the hassle to rebase . |
Add the the text from the release notes to the actual documentation
docs/tor.mdto document how users can preserve their old address.Just in case users would like for some reason to preserve their existing v2 address with a not recommended text.