Pass clientAddress to firewall

[gmaclennan]

Pass through the clientAddress (from @hyperswarm.dht) to the opts.firewall function.

[gmaclennan]

@mafintosh any chance of merging this?

[mafintosh]

Can you explain the use case when its only for remote connections?

[gmaclennan]

We use mdns for discovery on the local network (and cases where there is no internet connectivity). We advertise a different port on mdns (which doesn't connect through hyperswarm, but uses secretStream directly).

In order to avoid duplicate connections we want to block connections through hyperswarm from local IPs. It seems like the firewall is the best place to do this?

[mafintosh]

Ok, we can add it to fix your immediate use case - will land local discovery at some point, but as a middle ground don't document it so its not guaranteed to be there in the future.

[gmaclennan]

Want me to remove the README change in this PR?

[mafintosh]

ya

[gmaclennan]

Force-pushed to remove README changes

[sethvincent]

It looks like this method call may also need to be updated. Is that right?

hyperswarm/index.js

Line 139 in 80ea460

if (this._handleFirewall(peerInfo.publicKey, null)) {

@gmaclennan