holepunchto / dht-rpc

Make RPC calls over a Kademlia based DHT.
MIT License
196 stars 37 forks source link

dgram is deprecated #20

Closed DougAnderson444 closed 4 years ago

DougAnderson444 commented 4 years ago

https://www.npmjs.com/package/dgram

But it's used in dht-rpc

mafintosh commented 4 years ago

dgram is a core api provided by node.js

https://nodejs.org/dist/latest-v12.x/docs/api/dgram.html

DougAnderson444 commented 4 years ago

Hmm, ok. The reason I noted this was I was getting an error when trying to run hyperswarm about not being able to resolve dgram.

Module not found: Can't resolve 'dgram' in '\node_modules\dht-rpc'

I saw it's require'd in dht-rpc.

Since it's part of Node, does this have to be changed?

DougAnderson444 commented 4 years ago

Ah, wait, maybe it's only for v12 (and I'm running node 10)

mafintosh commented 4 years ago

It’s been in Node since the beginning. What platform are you on?

DougAnderson444 commented 4 years ago

Windows 8