dougmoscrop / serverless-http

Use your existing middleware framework (e.g. Express, Koa) in AWS Lambda 🎉
Other
1.72k stars 166 forks source link

fix originalUrl #111

Closed adrai closed 5 years ago

adrai commented 5 years ago

I compared that to the behavior of aws-serverless-express.

adrai commented 5 years ago

any feedback?

dougmoscrop commented 5 years ago

Well the tests fail - I don't mind working from here as a starting point but I just don't have the time at the moment.

adrai commented 5 years ago

You're right, I adapted the tests

adrai commented 5 years ago

no need to hurry... adapted my code to work with req.url in the meantime

dougmoscrop commented 5 years ago

some background on .requestContext.path - https://github.com/dougmoscrop/serverless-http/pull/42#issuecomment-376981255

I'm not sure about any of this, we're using API gateway behind a custom domain for all things.