swim icon indicating copy to clipboard operation
swim copied to clipboard

Published 20 hours ago • verified publisher icon swimos

Close header fix

Open DobromirM opened this issue 2 years ago • 1 comments

  • Moved close from didRequest to didRespond as per RFC 2616.
  • Fixed didRespond being called too early.

Fixes #100

DobromirM avatar Jan 12 '23 16:01 DobromirM

The changes to the AbstractHttpServer looks good.

The changes to the Station might impact the performance of the I/O with WebSockets. What is the behavior without this change?

didRespond for AbstractHttpServer was being called before the response was actually received by the client, resulting in a server error if the close method is called.

DobromirM avatar Jan 16 '23 11:01 DobromirM