awslabs / aws-c-http

C99 implementation of the HTTP/1.1 and HTTP/2 specifications
Apache License 2.0
136 stars 42 forks source link

WebSocket ref-counts its HTTP request #403

Closed graebm closed 1 year ago

graebm commented 2 years ago

The WebSocket wasn't using acquire()/release() ref-counting on its HTTP request, because this code was written before we realized ref-counting was a thing we needed to do.

Also, correct a bunch of comments about old-fashioned lifetime management that don't apply now that we ref-count everything.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.