fix: respect command ignores HTTP_PROXY#2805
Conversation
🦋 Changeset detectedLatest commit: bd37494 The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Co-authored-by: Dmytro Anansky <dmytro@redocly.com>
Co-authored-by: Dmytro Anansky <dmytro@redocly.com>
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 6443b3c. Configure here.
|
@harshit078 you have some errors with formatting, This should resolve the failed code style check. |
|
Hi @DmitryAnansky, Pushed a Fix for it in the last commit. Thanks ! |

What/Why/How?
withConnectionClient() function in index.tsas a BaseResolver which inreturn should fetchHTTP_PROXYand other related.Reference
#2668
Testing
Screenshots (optional)
Check yourself
Security
Note
Medium Risk
Changes network resolution behavior for remote source descriptions and external
$refs by wiring a newBaseResolverwith a custom fetch; misconfiguration could impact connectivity or proxy bypass (NO_PROXY) behavior. Covered by targeted unit tests asserting proxy/mTLS fetch selection.Overview
Fixes
respectso remote source descriptions and external$refresolution consistently honorHTTP_PROXY/HTTPS_PROXY/NO_PROXYby passing a dedicatedBaseResolverintorunthat uses the same proxy/mTLS-awarecustomFetch.Adds unit tests verifying
externalRefResolveris provided and that itscustomFetchswitches between bareundicifetch vs proxy-aware/mTLS-aware implementations based on env vars and--mtls. Includes a patch changeset documenting the fix.Reviewed by Cursor Bugbot for commit bd37494. Bugbot is set up for automated code reviews on this repo. Configure here.