freedomjs / freedom-port-control

Opens ports through a NAT with NAT-PMP, PCP, and UPnP
11 stars 3 forks source link

Optimized addMapping() #4

Closed kennysong closed 9 years ago

kennysong commented 9 years ago

Added a PortControl.protocolSupportCache object, which addMapping() now uses. UPnP addMapping() is optimized to skip SSDP if we pass in a control URL. UPnP deleteMapping() is simplified. PortControl.deleteMapping() methods now all consistently use the Mapping.deleter() method.

Review on Reviewable