Skip to content

feat(proxy): add outbound HTTP proxy support for OAuth + provider req…#22

Open
sodie2323 wants to merge 4 commits intodecolua:masterfrom
sodie2323:feat/outbound-proxy-cli
Open

feat(proxy): add outbound HTTP proxy support for OAuth + provider req…#22
sodie2323 wants to merge 4 commits intodecolua:masterfrom
sodie2323:feat/outbound-proxy-cli

Conversation

@sodie2323
Copy link
Contributor

@sodie2323 sodie2323 commented Jan 20, 2026

  • Patch Node fetch via undici ProxyAgent when HTTP_PROXY/HTTPS_PROXY/ALL_PROXY is set
  • Ensure proxy patch is loaded for both chat pipeline and OAuth token exchange
  • Add CLI entrypoint with --proxy/--no-proxy flags to start the server with proxy envs
  • Document proxy usage examples in README

@sodie2323 sodie2323 force-pushed the feat/outbound-proxy-cli branch 9 times, most recently from 4b1b0b1 to 2c36f97 Compare February 7, 2026 08:53
@sodie2323
Copy link
Contributor Author

image Synced to the latest upstream branch and added a web-based UI for configuration.

@sodie2323
Copy link
Contributor Author

If it looks good, please merge. Thanks! I’d really love to get this feature in

@ViriDendron
Copy link

@sodie2323 can you please resolve conflicts.
Thank you :-)

@sodie2323 sodie2323 force-pushed the feat/outbound-proxy-cli branch from b4171db to 9db28ad Compare February 25, 2026 05:08
@sodie2323
Copy link
Contributor Author

@sodie2323 can you please resolve conflicts. Thank you :-)

conflicts resolved. Thanks!

@ViriDendron
Copy link

@sodie2323 I am sorry but the PR has conflicts again 🙃

@decolua is it possible to merge this PR next? :-)

- Patch Node fetch via undici ProxyAgent when HTTP_PROXY/HTTPS_PROXY/ALL_PROXY is set
- Ensure proxy patch is loaded for both chat pipeline and OAuth token exchange
- Add Dashboard Settings → Network to edit outbound proxy and apply immediately
- Persist outbound proxy settings in local db and initialize on server startup
- Document proxy usage examples in README
@sodie2323 sodie2323 force-pushed the feat/outbound-proxy-cli branch from 9db28ad to d8312d0 Compare February 25, 2026 09:58
@sodie2323
Copy link
Contributor Author

@ViriDendron Done

@ViriDendron
Copy link

@decolua can you please merge the PR.
Thank you :-)

@ViriDendron
Copy link

@sodie2323 can you please fix the first word "…uests" in your PR ?

@sodie2323
Copy link
Contributor Author

@ViriDendron deleted,thanks for the reminder

@decolua
Copy link
Owner

decolua commented Feb 28, 2026

Hi @sodie2323

We've cherry-picked and merged your PR into our main branch. Thank you for the contribution — the outbound proxy support is a great addition, especially the clean separation into src/lib/network/ and the no-restart apply behavior.

We also applied two small fixes on top:

  • Close old ProxyAgent dispatcher when the proxy URL changes (prevent connection pool leak)
  • Add idempotency guard on globalThis.fetch patch to avoid wrapping multiple times

Appreciate the work!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants