Skip to content

Conversation

sipa
Copy link
Member

@sipa sipa commented Feb 25, 2012

Before 0.6 addrProxy was a CAddress, but netbase changed it to CService.
Retain compatibility by wrapping/unwrapping with a CAddress when saving
or loading.

This commit retains compatibility with 0.6.0rc1 (which wrote the setting
as a CService) by trying to parse twice.

Fixes #814

Before 0.6 addrProxy was a CAddress, but netbase changed it to CService.
Retain compatibility by wrapping/unwrapping with a CAddress when saving
or loading.

This commit retains compatibility with 0.6.0rc1 (which wrote the setting
as a CService) by trying to parse twice.
@sipa
Copy link
Member Author

sipa commented Feb 25, 2012

In case compatibility with 0.6.0rc1 is not required, this code suffices:

if (strKey == "addrProxy") { CAddress addr; ssValue >> addr; addrProxy = addr; }

@gavinandresen
Copy link
Contributor

ACK.

@gavinandresen gavinandresen merged commit 4a10d4c into bitcoin:master Feb 27, 2012
suprnurd pushed a commit to chaincoin-legacy/chaincoin that referenced this pull request Dec 5, 2017
lateminer pushed a commit to lateminer/bitcoin that referenced this pull request Oct 30, 2019
a789d21 [zPIV] Disable zerocoins inclusion in 'SelectStakeCoins' (random-zebra)

Tree-SHA512: 92da55ee6efe3090ad1b279e3ad4d1f757ca70d0a4b1f820a36b4d5d6dcef126002fc6c2402b2b8d34618f212a192d8dc7505f98a0050691d152ba5a17fb1821
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

IPv6 address in proxy field?
2 participants