In HTTP CONNECT, use hostname parsed from TLS SNI or HTTP Host header (renew #71)

[andronoob]

This renews #71

[andronoob]

Encountered this error:

redsocks: redsocks.c:1165: redsocks_relay_connected: Assertion `buffev == client->relay' failed.
Aborted (core dumped)

This happens while POSTing data.

Update: fixed this in c902860

[andronoob]

Oh this patch is was also affected by this issue: nodejs/http-parser#501

Update: fixed this in 809aa9c

[andronoob]

Protocols which are neither plaintext HTTP nor TLS are dropped. Something like "Invalid HTTP method" should not be a fatal error.

Fixed in 5755174

[andronoob]

Bad news, seemed to be double free memory corruption.😕

free(): invalid next size (fast)
Aborted (core dumped)

Still don't know how to reproduce this.

Update: this should be fixed in 9b0a5da

[tdferreira]

@darkk there's nobody to review this?
It would be a must to have this feature.