Fix coalesced handshake hashing, again

drogue-iot / embedded-tls

An Rust TLS 1.3 implementation for embedded devices.

Apache License 2.0

168 stars 21 forks source link

Fix coalesced handshake hashing, again #118

Closed bugadani closed 1 year ago

bugadani commented 1 year ago

Unfortunately, rustls doesn't coalesce handshake messages, or at least not in our current test suite. This meant that the last refactor (#114) silently broke #75 again. This PR fixes the issue.

lulf commented 1 year ago

Ah :( Netty does coalesce. I'll try to get a test with that going

bugadani commented 1 year ago

A unit test would have prevented this, too.