Rather than doing a circle and re-resolving the node's IP, just use the one from nodestats directly.
This requires syncing the addr field from CNode.
Fixes #8876
gui: fix ban from qt console #8885
pull theuni wants to merge 1 commits into bitcoin:master from theuni:fix-gui-ban changing 3 files +15 −12-
theuni commented at 11:42 PM on October 4, 2016: member
-
cb78c60534
gui: fix ban from qt console
Rather than doing a circle and re-resolving the node's IP, just use the one from nodestats directly. This requires syncing the addr field from CNode.
- fanquake added the label GUI on Oct 5, 2016
- laanwj cross-referenced this on Oct 5, 2016 from issue Multiple Selection for peer and ban tables by achow101
-
MarcoFalke commented at 6:54 PM on October 6, 2016: member
Can confirm that cb78c60534e5be205f9190cb0cde700f9e9fa38d fixes the issue.
-
laanwj commented at 11:05 AM on October 8, 2016: member
ut/code review ACK cb78c60
- jonasschnelli cross-referenced this on Oct 8, 2016 from issue Multiple peer operations via GUI by ghost
- jonasschnelli approved
-
jonasschnelli commented at 2:42 PM on October 8, 2016: contributor
Tested ACK 7f0b651ac9ec27e0bfda399146d0ceb527456160
- jonasschnelli merged this on Oct 8, 2016
- jonasschnelli closed this on Oct 8, 2016
- jonasschnelli referenced this in commit b2fec4e9a1 on Oct 8, 2016
- codablock referenced this in commit 5a588b00af on Sep 7, 2017
- UdjinM6 referenced this in commit 9707ca5cea on Sep 9, 2017
- bitcoin locked this on Sep 8, 2021
