-
Notifications
You must be signed in to change notification settings - Fork 25
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Error using ollama through a proxy #48
Comments
@r0man can you take a look at this? |
Hi @theasp, which proxy are you using? I tried to re-produce this with tinyproxy but my request seems to work. Could you please run this:
and send me |
Hi @r0man, it's an old version of Squid. I'm not the one that maintains it sadly. Based on the output below, I suspect that This is response.txt when using the proxy:
This is without the proxy:
|
@theasp The support for proxies was missing. I added this here: r0man/plz-media-type#8 But now I'm running into another issue:
I will see what I can do .... |
@theasp Well, I think the remaining issue is with the event source and OpenAI. But since you use Ollama, could you try evaluating this file: And see if it is working for you? I don't have Ollama setup via an HTTPS proxy. But I'm using this now with Vertex and it seems to be working. |
@r0man that seems to be working fine for me, thanks! |
@theasp Ok, perfect. Thanks for trying! I will open a PR here when I got the event source and OpenAI working as well. |
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: - r0man/plz-media-type#8 - r0man/plz-event-source#5 It fixes the issue ahyatt#48 reported by @theasp. He tested the code with Ollama behind a Squid proxy. I tested it with Vertex and OpenAI behind a tinyproxy. ahyatt#48
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: - r0man/plz-media-type#8 - r0man/plz-event-source#5 It fixes the issue ahyatt#48 reported by @theasp. He tested the code with Ollama behind a Squid proxy. I tested it with Vertex and OpenAI behind a tinyproxy. ahyatt#48
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: - r0man/plz-media-type#8 - r0man/plz-event-source#5 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. #48
When this library is used with a proxy (
http_proxy
andhttp_proxy
env variables defined) it encounters an error dealing with the HTTP header regex. This does not happen when not using the proxy, andplz
seems to have no problem accessing it. It looks like it's an issue withplz-media-type
to me.The text was updated successfully, but these errors were encountered: