This adds support for HTTP/HTTPS proxies. The code that was dealing with this in plz.el wasn't migrated and we did not had any tests for this. This should now be fixed. See:
It fixes the issue #48 reported by @theasp. He tested the code with Ollama behind a Squid proxy. I tested it with Vertex and OpenAI behind a tinyproxy.
It also removes the default plz-timeout, since this is going to be removed in the next version of plz. It should not affect us, since we already have a default that is set to nil.
ahyatt
commented
1 month ago
This adds support for HTTP/HTTPS proxies. The code that was dealing with this in plz.el wasn't migrated and we did not had any tests for this. This should now be fixed. See:
It fixes the issue #48 reported by @theasp. He tested the code with Ollama behind a Squid proxy. I tested it with Vertex and OpenAI behind a tinyproxy.
https://github.com/ahyatt/llm/issues/48
It also removes the default
plz-timeout, since this is going to be removed in the next version of plz. It should not affect us, since we already have a default that is set to nil.