postcss-import-url icon indicating copy to clipboard operation
postcss-import-url copied to clipboard

Feature: support proxies

Open leepowelldev opened this issue 5 years ago • 5 comments

Sadly this package currently fails to work for us behind a corporate proxy as http-https fails to pick up proxy settings from environment variables on the OS. The request package does this very well - so this is a simple swap out from http-https to request. Interface remains unchanged.

leepowelldev avatar Sep 23 '20 09:09 leepowelldev

DeepCode's analysis on #f0926c found:

  • :warning: 1 warning :point_down:

Top issues

Description Example fixes
No catch method for promise. This may result in an unhandled promise rejection. Occurrences: :wrench: Example fixes

👉 View analysis in DeepCode’s Dashboard | Configure the bot

ghost avatar Sep 23 '20 09:09 ghost

I think we need to choose another replacement, request is deprecated.

unlight avatar Sep 23 '20 14:09 unlight

Ah, not sure how I missed that. I’ll investigate alternatives.

On 23 Sep 2020, at 15:54, Roman Vasilev [email protected] wrote:

 I think we need to choose another replacement, request is deprecated.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

leepowelldev avatar Sep 23 '20 16:09 leepowelldev

https://github.com/sindresorhus/awesome-nodejs#http https://github.com/mikeal/r2

unlight avatar Sep 23 '20 17:09 unlight

Thanks. I was thinking of looking at axios. It’s fairly small and is well maintained.

leepowelldev avatar Sep 23 '20 17:09 leepowelldev