Add TLS SNI patch

Open Moult opened this issue 6 years ago • 4 comments

May 03 '19 12:05 Moult

What frees memory allocated by get_socket_host_name? What when this function returns NULL? What if mem_alloc returns NULL around line 356? In this case is memleak for sure.

May 03 '19 13:05 rkd77

Sorry @rkd77 I'm not a C programmer, so I couldn't answer. I'm only applying the patch found here:

https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=797968

I would really appreciate it if you could work it out, since having SNI support is very important to view a lot of websites nowadays :)

May 05 '19 00:05 Moult

Sorry, I just cloned the latest master and tried recompiling, maybe this PR is not required. Maybe TLS SNI already works.

May 05 '19 04:05 Moult

According to https://check-tls.akamaized.net/, the valid TLS SNI is present in the request.

Feb 10 '20 13:02 hellricer