Remove I2P support from netbase #1913

pull sipa wants to merge 1 commits into bitcoin:master from sipa:noi2p changing 2 files +6 −41

sipa commented at 1:10 PM on October 7, 2012: member

I2P apparently needs 256 bits to store a fully routable address. Garlicat requires a centralized lookup service to map the 80-bit addresses to fully routable ones (as far as I understood), so that's not really usable in our situation.

To support I2P routing and peer exchange for it, another solution is needed. This will most likely imply a network protocol change, and extension of the 'addr' message.

Remove it for now, until we can support it completely.

Remove I2P support from netbase

I2P apparently needs 256 bits to store a fully routable address. Garlicat
requires a centralized lookup service to map the 80-bit addresses to fully
routable ones (as far as I understood), so that's not really usable in our
situation.

To support I2P routing and peer exchange for it, another solution is needed.
This will most likely imply a network protocol change, and extension of the
'addr' message.

4e882b7960

Diapolo commented at 4:09 PM on October 7, 2012: none

This is the right thing to do and if it causes no compilation problems it should get into 0.7.1 IMO. We don't need unused / incpomplete stuff in the code.
jgarzik commented at 9:57 PM on October 8, 2012: contributor

ACK
laanwj commented at 5:27 AM on October 9, 2012: member

ACK, no need to keep half-finished stuff in
laanwj referenced this in commit 24694e24ed on Oct 11, 2012
laanwj merged this on Oct 11, 2012
laanwj closed this on Oct 11, 2012
pyritepirate referenced this in commit cda91bf8fa on Jan 23, 2019
KolbyML referenced this in commit 0a820797fb on Dec 5, 2020
DrahtBot locked this on Sep 8, 2021

Contributors