-
-
Notifications
You must be signed in to change notification settings - Fork 1k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Q: use case for caching proxy #137
Comments
Thanks for opening a new issue. The team has been notified and will review it as soon as possible. |
I think
Supporting TLS is a gigantic feature for |
ok, thanks for the info. i know tidwall said that he does not support tls because core tls library depends on net.Conn, iirc. if the tls "plugin" would be a simple ReadWriter like compressors and marshallers can be, that would be the most universal solution, i think. i will probably look into it once i begin the work, though it is highly possible that due to this i will have to take the go std route for now and look at gnet for v2 later on. since getting the working solution out as fast as possible will be priority. |
actually, i have a better idea. it might sound a bit too crazy but i think it would make more sense to just fork Go and simply replace the std net package's internals with gnet. That way there would be no compatibility issue and no need to write tls/http/http2/http3/quic plugins. and maybe in time it could be merged into Go if it proves to be "worthy". |
I don't think that will work, |
ok, you know best. thanks for the info. |
I will be making a caching proxy soon where user sends a HTTP/S request and i either serve the locally cached content or fetch it from origin, cache it and serve it. So I am wondering if this would be a good library to use instead of built-in net package to yield some more performance?
Secondly, HTTPS/TLS is a must and I saw that you do not use tidwall's evio, at least not any more, which does not support TLS. But also this looks like #16 TLS might still not be supported here, so what is the situation?
The text was updated successfully, but these errors were encountered: