bermi / sauce-connect-launcher

Downloads and launches an instance of Sauce Connect Selenium Proxy https://wiki.saucelabs.com/display/DOCS/Sauce+Connect+Proxy
72 stars 68 forks source link

Feature request: use Npm download mecanism instead of Http request #52

Closed jmorille closed 8 years ago

jmorille commented 9 years ago

Like discut in

Not use Http request to download plugins but use the npm registry mecanim by putting them in the heroku registry.

That will permit to secure, cache and accelerate the installation procedure (thank to enterprise npm registry).

That permit to be used in company with Business Continuity Plan (BCP) constraint or behind a corporate firewall.

bermi commented 9 years ago

@jmorille Would this imply that we'd be on the hook for keeping a copy of SC on npm? Would it work for you to specify the path to sauce connect via an environment variable and skip the download process?

sebv commented 9 years ago

I guess it'll be best to get rid of the post-install trigger, and move it to a custom script. To do that using an environment variable is not great.

jmorille commented 9 years ago

sorry for my late response, I come back from vacancies...

I would like that the download mecanism use the npm configuration, in order to pass by me corporate proxy cache (Sonatype Nexus), with respect of NPM download convention, to have a local cache. THe actual mecanism use a direct download, that don't pass the proxy configuration and not use the local Npm registry.

sonnyking commented 9 years ago

Yes, this issue is preventing a fortune 100 enterprise from adopting WCT, hundreds of developers.

mdwragg commented 9 years ago

@sonnyking :+1:

Jonahss commented 9 years ago

Ah I see. Sounds like a good thing to fix.

panuhorsmalahti commented 8 years ago

:+1:

johanneswuerbach commented 8 years ago

Closing as sauce-connect-launcher supports http proxies now: https://github.com/bermi/sauce-connect-launcher/pull/80