diff options
author | Alexander Mihajlovic <[email protected]> | 2019-03-01 20:18:37 +0100 |
---|---|---|
committer | Loïc Hoguin <[email protected]> | 2019-04-25 10:32:12 +0200 |
commit | c35d399138e99d3615a3ceac40a2440030d9a2ef (patch) | |
tree | 429db83664be6b2d1e780d34c6b823d09fa70ef7 /test/ws_autobahn_SUITE.erl | |
parent | 561847bb14e91e20e4e5f0106d33d693cd575498 (diff) | |
download | gun-c35d399138e99d3615a3ceac40a2440030d9a2ef.tar.gz gun-c35d399138e99d3615a3ceac40a2440030d9a2ef.tar.bz2 gun-c35d399138e99d3615a3ceac40a2440030d9a2ef.zip |
Remove explicit call to dbg:start/0
This fixes a bug in `gun` when trying to open 2 connections with tracing enabled.
You can recreate it like so:
> application:ensure_all_started(gun).
> gun:open("localhost", 80, #{trace => true}).
> gun:open("localhost", 80, #{trace => true}).
The second call fails with a cause clause exception.
You'll observe the same exception by:
> dbg:start().
> dbg:start().
Ultimately this may be a bug in OTP, but since `dbg:start/0` is undocumented
I'm inclined to give it the benefit of the doubt. Regardless, since the docs[1]
clearly state that calling `dbg:tracer/0` is the proper way to start the default
tracer message receiver, I argue that calling `dbg:start/0` is a bug in `gun`.
[1]: http://erlang.org/doc/man/dbg.html
Diffstat (limited to 'test/ws_autobahn_SUITE.erl')
0 files changed, 0 insertions, 0 deletions