Skip to content
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

fix request should failed when invaild proxy provided #9

Merged
merged 1 commit into from
Dec 30, 2021

Conversation

msivasubramaniaan
Copy link
Contributor

The PR fix supports the http request should throw error when invalid proxy configured.

This problem found by users of yaml-language-server, issue: redhat-developer/yaml-language-server#588

@ghost
Copy link

ghost commented Dec 21, 2021

CLA assistant check
All CLA requirements met.

@msivasubramaniaan
Copy link
Contributor Author

@aeschli Could you look on this?

@aeschli aeschli merged commit a6dc124 into microsoft:main Dec 30, 2021
@aeschli
Copy link
Contributor

aeschli commented Dec 30, 2021

Thanks a lot! Looks like the agent support got lost with 62f6c36
Thanks for fixing this and adding a test.
The tests causes timeouts on my machine so I replaced it with other asserts: fa2fb79

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.

2 participants