search

NFIBrokerage / slipstream

A slick WebSocket client for Phoenix Channels
https://hex.pm/packages/slipstream
Apache License 2.0
160 stars 18 forks source link

add :telemetry for connections #13

Closed the-mikedavis closed 3 years ago

the-mikedavis commented 3 years ago

shouldn't be widely used, but should be useful during these early days of using slipstream and catching rogue :gun messages

the-mikedavis commented 3 years ago

each c:GenServer.handle_info/2 clause in Slipstream.Connection should wrap its operation in a :telemetry.span/3

in order to make this easier and less repetitive, it might make sense to refactor the connection process to use the token pattern, so that everything that it does can be retrospected when the :telemetry.span/3 is ready to be done

the-mikedavis commented 3 years ago

added in v0.3.0