Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

mocknet newStream notification issue #31

Closed
whyrusleeping opened this issue Apr 2, 2016 · 2 comments
Closed

mocknet newStream notification issue #31

whyrusleeping opened this issue Apr 2, 2016 · 2 comments
Labels
exp/wizard Extensive knowledge (implications, ramifications) required help wanted Seeking public contribution on this issue kind/bug A bug in existing code (including security flaws)

Comments

@whyrusleeping
Copy link
Contributor

whyrusleeping commented Apr 2, 2016 •
edited

When opening a new stream using the mocknet (p2p/net/mock) the connection handler doesnt get called until a write actually occurs. I beleive this is a bug.
@whyrusleeping whyrusleeping added the help wanted Seeking public contribution on this issue label Sep 14, 2016
@ghost ghost added status/deferred Conscious decision to pause or backlog and removed status/deferred Conscious decision to pause or backlog labels Nov 12, 2016
@whyrusleeping whyrusleeping added the status/deferred Conscious decision to pause or backlog label Nov 28, 2016
@daviddias daviddias added kind/bug A bug in existing code (including security flaws) exp/wizard Extensive knowledge (implications, ramifications) required labels Aug 19, 2017
@Stebalien
Copy link
Member

Is this still the case? This looks fixed.

@anacrolix
Copy link
Contributor

Let's close this due to age and lack of activity.

@ghost ghost removed the status/deferred Conscious decision to pause or backlog label Feb 19, 2019
marten-seemann pushed a commit that referenced this issue Apr 22, 2022
@Stebalien
Merge pull request #31 from libp2p/feat/refactor
c1eeb0c 
fix for transport refactor
marten-seemann pushed a commit that referenced this issue Apr 22, 2022
@Stebalien
Merge pull request #31 from libp2p/update-deps
7ff218a 
update to the latest quic-go, gx publish
marten-seemann added a commit that referenced this issue Apr 22, 2022
@marten-seemann
Merge pull request #31 from libp2p/update-yamux
f3029b6 
update go-yamux to v2.0.0, use context passed to OpenStream
marten-seemann added a commit that referenced this issue Apr 25, 2022
@marten-seemann
update mplex (#31)
d6fc4be
marten-seemann added a commit that referenced this issue Apr 26, 2022
@marten-seemann
Merge pull request #31 from libp2p/web3-bot/sync
386648e 
sync: update CI config files
marten-seemann added a commit that referenced this issue May 18, 2022
@marten-seemann
Merge pull request #31 from libp2p/open-stream-context
a3d1008 
pass contexts to OpenStream in tests
marten-seemann pushed a commit that referenced this issue Jul 1, 2022
@Stebalien
Merge pull request #31 from libp2p/fix/reset-test-race
3f15e2c 
fix race in reset test
marten-seemann pushed a commit that referenced this issue Aug 17, 2022
Merge pull request #31 from libp2p/bug/remove-blowfish-support
20123ca 
Remove support for blowfish
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
exp/wizard Extensive knowledge (implications, ramifications) required help wanted Seeking public contribution on this issue kind/bug A bug in existing code (including security flaws)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@Stebalien @anacrolix @daviddias @whyrusleeping