Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hey, so I discovered this module when I was trying to write my own pull-stream wrapper for
net
and was very happy, but not everything was working as I hoped. So I started changing and fixing some things as well as I could. I probably made a lot of mistakes so please forgive them.Currently this adds
client.js
as this worked out very nicely in my testsipv6
&ipv4
support when dialing and listeningclose
method usingpull-abortable
to sockets created fromhandle.js
standard
(I was a bit confused by the code style so please let me know if you want me to remove this/move to a different code style)Currently the main issue I'm facing is that when I use the
close
method it's not closing properly the other side any tips on how to move forward would be appreciated.A test suite testing these features and showing this problem can be found here: libp2p/js-libp2p-tcp#20 which needs the changes in this PR and afterwards can be run with
npm test
.