localstack / serverless-localstack

⚡ Serverless plugin for running against LocalStack
519 stars 83 forks source link

Strip endpoint hostname trailing slash #229

Closed kristallizer closed 8 months ago

kristallizer commented 1 year ago

Not stripping the trailing slash leads to malformed URLs in the output:

Before: https://7d7c2bf6.execute-api.localhost.localstack.cloud:4566//local/task After: https://7d7c2bf6.execute-api.localhost.localstack.cloud:4566/local/task

The right choice here is dependent on how you define the path in serverless.yml. Seeing how the official docs for API Gateway v2 add a / before the path, I propose that the trailing slash should be stripped here.

steffyP commented 8 months ago

Hi @kristallizer, your fix was released with serverless-localstack version 1.2.0 🚀