Skip to content
This repository was archived by the owner on Feb 7, 2024. It is now read-only.

PubSub tests hang with go-ipfs master #62

Closed
ghost opened this issue Nov 20, 2017 · 1 comment
Closed

PubSub tests hang with go-ipfs master #62

ghost opened this issue Nov 20, 2017 · 1 comment
Assignees
Labels
kind/bug A bug in existing code (including security flaws)

Comments

@ghost
Copy link

ghost commented Nov 20, 2017

Example: https://travis-ci.org/ipfs/go-ipfs-api/jobs/304527816

This seems to have started with the merge of ipfs/kubo#3856.

I changed the t.Log() calls to fmt.Printf() locally, and saw that it's hanging at subscribing..., and Wireshark tells me that the API doesn't bother sending response header until the first message comes in (which I emulated with ipfs pubsub pub test foobar).

=== RUN   TestAdd
--- PASS: TestAdd (0.00s)
=== RUN   TestLocalShell
--- PASS: TestLocalShell (0.00s)
=== RUN   TestCat
--- PASS: TestCat (0.00s)
=== RUN   TestList
--- PASS: TestList (0.00s)
=== RUN   TestFileList
--- PASS: TestFileList (0.00s)
=== RUN   TestPins
--- PASS: TestPins (0.01s)
=== RUN   TestPatch_rmLink
--- PASS: TestPatch_rmLink (0.00s)
=== RUN   TestPatchLink
--- PASS: TestPatchLink (0.00s)
=== RUN   TestResolvePath
--- PASS: TestResolvePath (0.00s)
=== RUN   TestPubSub
subscribing...
@ghost ghost added the kind/bug A bug in existing code (including security flaws) label Nov 20, 2017
@ghost ghost assigned keks Nov 20, 2017
@ghost
Copy link
Author

ghost commented Nov 22, 2017

Fixed in #63 and ipfs/kubo#4402

@ghost ghost closed this as completed Nov 22, 2017
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
kind/bug A bug in existing code (including security flaws)
Projects
None yet
Development

No branches or pull requests

1 participant