5.1.2 no longer supports HTTP proxy

[nejch]

Just to confirm #402 (comment), 5.1.2 no longer downloads the go binary via proxy:

$ yarn lint:editorconfig
yarn run v1.22.19
$ editorconfig-checker docs/*/*.md docs/*.md *.md
Failed to download binary:
HttpError: fetch failed
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

(follow-up to #394)

We re-added this earlier this year in #395, so might just need some tweaking.

/cc @theoludwig

[theoludwig]

Thanks for creating the issue! @nejch

I don't know how I could test it myself, I don't have a HTTP Proxy to test.
And the code haven't changed much in relation of ProxyAgent and related, only dependencies update, might be related that we updated proxy-agent and related dependencies.

Would be much appreciated if someone could:

1. Add a automated test so it doesn't happen again, and we do changes "fearlessly" (or at least less in the blind)
2. Fix the problem.

[theoludwig]

Do you think, it's related to --minify with vercel/ncc?

[nejch]

I'll try to take a look when I find some time. I was considering adding those in the initial PR but I think there were no tests yet at the time.

I agree it could be something to do with the major bumps in the agent library.

[github-actions]

🎉 This issue has been resolved in version 5.1.3 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀