search

najamelan / ws_stream_tungstenite

Provide AsyncRead/AsyncWrite over tungstenite websockets
31 stars 16 forks source link

Update tungstenite to 0.17.2 #9

Closed silvanshade closed 2 years ago

silvanshade commented 2 years ago

This PR just updates the tungstenite dependency and fixes the code per the API changes. As noted in the TungMessage::Frame branch in the code below, TungMessage::Frame should never occur from a read but is currently only intended to be used for fragmentated writes. This is discussed here.

najamelan commented 2 years ago

Hi, thanks, I will look into this tomorrow.

codecov[bot] commented 2 years ago

Codecov Report

Merging #9 (d1d4465) into dev (f165133) will decrease coverage by 0.14%. The diff coverage is 0.00%.

@@            Coverage Diff             @@
##              dev       #9      +/-   ##
==========================================
- Coverage   80.07%   79.92%   -0.15%     
==========================================
  Files          15       15              
  Lines         552      553       +1     
==========================================
  Hits          442      442              
- Misses        110      111       +1
Impacted Files Coverage Δ
src/tung_websocket.rs 59.83% <0.00%> (-0.50%) :arrow_down:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

najamelan commented 2 years ago

This looks good, thanks for the contribution.

najamelan commented 2 years ago

Version 0.8.0 has been released.