Error decoding fragmented frames

[jeanparpaillon]

See #70

[fhunleth]

Did you mean to include the decode_frame -> decode_header change?

Also, wow, nice find!

[jeanparpaillon]

@fhunleth if you don't mind, we can include the function name change, but you can pick the second commit only, if you prefer

btw, I would have liked adding a non-regression test (I have some sample data for this), but I was unable to run the tests. Got this error:

$ rebar3 ct 
===> Verifying dependencies...
===> Compiling websocket_client
src/websocket_client.erl:134:10: Warning: http_uri:parse/2 is deprecated and will be removed in OTP 25; use uri_string functions instead

src/wsc_lib.erl:7:2: Warning: export_all flag enabled - all functions will be exported

build/test/lib/websocketclient/test/wsreq_SUITE.erl:3:2: Warning: export_all flag enabled - all functions will be exported

build/test/lib/websocketclient/test/wsc_lib_SUITE.erl:3:2: Warning: export_all flag enabled - all functions will be exported

===> Running Common Test suites...
===> Error running tests:
  "Failed to start CTH, see the CT Log for details"

[fhunleth]

Sorry - I don't maintain this project. I just saw the fix and was trying to be helpful to get it merged as quickly as possible.

[sanmiguel]

I am finally making my way back to get this library up-to-date. Will pick up this PR soon.... thanks Jean!

[jeanparpaillon]

@sanmiguel you're welcome 🙏

[sanmiguel]

The PR contains unrelated changes that I don't want to accept, and the proposed fix for the issue causes other further problems with decoding in more general cases so I can't accept this as-is. Also, 4 years has passed, I'm guessing you've moved on to other things and this isn't causing issues any more either way. Closing for now, please reopen if you'd like me to revisit this.