Add support for http proxy

claudiajs / claudia

Deploy Node.js projects to AWS Lambda and API Gateway easily

https://claudiajs.com

MIT License

3.8k stars 274 forks source link

Add support for http proxy #117

Closed jason-riddle closed 7 years ago

jason-riddle commented 7 years ago

This should resolve https://github.com/claudiajs/claudia/issues/97. Two questions.

Where should the tests be added for this change?
If --aws-client-timeout and --proxy are both used, will one setting clobber the other? I ask since they set AWS.config.httpOptions.

gojko commented 7 years ago

thanks for this. I've updated the code so it lints OK and both proxy and timeout can work. I don't have a way of testing it locally, so can you please just pull from master and try it out, then I'll publish to NPM if OK.

jason-riddle commented 7 years ago

Cool, sounds good. Thanks!

gojko commented 7 years ago

this is now on NPM as 2.12.0.