holepunchto / hyperdht

The DHT powering Hyperswarm
https://docs.holepunch.to
MIT License
323 stars 46 forks source link

remove node net module #66

Closed heapwolf closed 2 years ago

heapwolf commented 2 years ago

After we get the new utp module done, we should remove net module to reduce size/complexity. It's only used in a few places.

%grep "require('net" -R . 
./node_modules/bind-easy/test.js:const net = require('net')
./node_modules/bind-easy/index.js:const net = require('net')
./node_modules/@hyperswarm/dht/test/connections.js:const net = require('net')
./node_modules/@hyperswarm/dht/lib/socket-pairer.js:const net = require('net')
./node_modules/@hyperswarm/secret-stream/test.js:const net = require('net')
kasperisager commented 2 years ago

I plan on tackling this in #82.

mafintosh commented 2 years ago

in master