Page MenuHomePhabricator
Differential D9188

net: CNetAddr: add support to (un)serialize as ADDRv2
ClosedPublic
Actions

Authored by Fabien on Feb 9 2021, 12:33.

Details

Reviewers
majcosta
Group Reviewers
Restricted Project
Commits
rABCc78ecba78410: net: CNetAddr: add support to (un)serialize as ADDRv2
Summary

Partial backport (3/4) of core PR19845:
https://github.com/bitcoin/bitcoin/pull/19845/commits/e0d73573a37bf4b519f6f61e5678572d48a64517

Depends on D9186 and D9187.

Test Plan
ninja all check-all

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Fabien created this revision.Feb 9 2021, 12:33
Herald added a reviewer: Restricted Project. · View Herald TranscriptFeb 9 2021, 12:33
Fabien requested review of this revision.Feb 9 2021, 12:33
teamcity edited the summary of this revision. (Show Details)Feb 9 2021, 12:34
Fabien added a child revision: D9189: net: recognize TORv3/I2P/CJDNS networks.Feb 9 2021, 12:36
Harbormaster completed remote builds in B15150: Diff 27602.Feb 9 2021, 13:01
majcosta accepted this revision.Feb 9 2021, 15:39
majcosta added a subscriber: majcosta.
majcosta added inline comments.
src/version.h
43–45 ↗(On Diff #27602)

We don't have SERIALIZE_TRANSACTION_NO_WITNESS

This revision is now accepted and ready to land.Feb 9 2021, 15:39
Closed by commit rABCc78ecba78410: net: CNetAddr: add support to (un)serialize as ADDRv2 (authored by Vasil Dimov <[email protected]>, committed by Fabien). · Explain WhyFeb 9 2021, 17:04
This revision was automatically updated to reflect the committed changes.
Fabien added a commit: rABCc78ecba78410: net: CNetAddr: add support to (un)serialize as ADDRv2.

Revision Contents
Changeset List

PathSize
src/
136 lines
57 lines
test/
200 lines
3 lines

Diff 27618

View Options

src/netaddress.h

Loading...
View Options

src/netaddress.cpp

Loading...
View Options

src/test/net_tests.cpp

Loading...
View Options

src/version.h

Loading...