-
Notifications
You must be signed in to change notification settings - Fork 108
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
feat: enable proxy handling for native fetch and connectrpc calls
Setting the proxy relies on undici for now undici does not respect the HTTP_PROXY vars, it's planned to build it in from v23 on by default: nodejs/node#43187 (comment) > My current plan is to add support for the HTTP_PROXY env variable behind a flag in v22 (and possibly v20), then unflag in v23. Using global dispatcher is the current way to go for native fetch, therefore undici needs to be added. For connectrpc the agent will be passed down the chain: /** * Options passed to the request() call of the Node.js built-in * http or https module. * example proxy trace: 22:05:23 HTTPS POST localhost/auth/realms/cosmo/protocol/openid-connect/token 200 application/json 2.0k 202ms 22:05:35 HTTP POST localhost/wg.cosmo.platform.v1.PlatformService/DeleteFederatedGraph 200 application/proto 44b 64ms
- Loading branch information
1 parent
030dc6d
commit 380d8e1
Showing
3 changed files
with
21 additions
and
2 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters