Skip to content

Honor https_proxy environment variables#424

Merged
enesgules merged 2 commits intoupstash:masterfrom
sregister:add-proxy
Aug 21, 2025
Merged

Honor https_proxy environment variables#424
enesgules merged 2 commits intoupstash:masterfrom
sregister:add-proxy

Conversation

@sregister
Copy link
Contributor

This should allow the mcp server to operate behind http proxy servers. If the standard https_proxy environment variable is defined, it should be used for requests to https://context7.com/api

@enesgules
Copy link
Collaborator

Would this solve #250 ?
Also could you resolve the conflicts please?

@sregister
Copy link
Contributor Author

Yes I think this should fix the problem described in #250

@enesgules
Copy link
Collaborator

I guess the bun lockfile needs to be updates as well for the workflow to run successfully

@sregister
Copy link
Contributor Author

I guess the bun lockfile needs to be updates as well for the workflow to run successfully

I think this should be fixed now

@enesgules
Copy link
Collaborator

enesgules commented Aug 9, 2025

Could you update the readme to document this proxy support please, also please run the linter before your final commit

If standard https_proxy environment variables exist, make proxied
requests to https://context7.com/api
@sregister
Copy link
Contributor Author

I've added a note to README that standard https_proxy environment variables should be used.

@tricktron
Copy link

@sregister @enesgules I resolved the conflict on my fork and can confirm that it works behind our corporate proxy.

@enesgules
Copy link
Collaborator

Awesome @tricktron thanks for checking, will merge today

Copy link
Collaborator

@enesgules enesgules left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TYSM!

@enesgules enesgules merged commit e924153 into upstash:master Aug 21, 2025
1 check failed
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