Remove send to IP address and IP transactions support #253

Remove send to IP transaction support, as it is insecure and would confuse users when used.

An IP is a terrible identifier as it does not identify a person or organization. Furthermore, man-in-the-middle attacks are trivial as the internet has no "proof that you have ip XXX" API. In the future address shortening services based on "user@domain" would be useful, but these will likely be based on third party APIs and not on this code.

• Removes logic to accept transfers by IP
• Removes UI logic to send transfers by IP (CSendingDialog). Entering an IP in the send box will always result in an error.

Discussion on forum: https://forum.bitcoin.org/index.php?topic=9334.0

ACK

and I think Gavin is in favor of this, too

NACK, it would appear code was ripped out of uibase.* without modifying the uiproject.fpb file meaning no one can edit the wxUI in the future.

Also, please update for latest changes (lots!). We wanted to pull wallet crypto first.

yeah I will, first have to get this wx dialog editor tooling

Aww. The removal of this would mean we can't run a lottery for nodes running the most recent version of bitcoin as an incentive to make people upgrade. :)

Wow, you probably hit upon the first actual useful application for send to IP transactions. But watch out for people MITMing you to steal the incentives :)

Well, if they are going to go through the trouble of setting up something that pretends to be a current node it would be easier to actually RUN a current node. The lottery could do some reasonable health checks too. :) Like make sure the winner can feed the block chain and forward a transaction.