Fix URL port/port when default protocols - Githubissues

awslabs / llrt

LLRT (Low Latency Runtime) is an experimental, lightweight JavaScript runtime designed to address the growing demand for fast and efficient Serverless applications.

Apache License 2.0

7.74k stars 342 forks source link

Fix URL port/port when default protocols #321

Closed richarddavison closed 3 months ago

richarddavison commented 3 months ago

Issue # (if available)

Fixes https://github.com/awslabs/llrt/issues/320

Description of changes

Port should be removed if default for the protocol

Checklist

[x] Created unit tests in tests/unit and/or in Rust for my feature if needed
[x] Ran make fix to format JS and apply Clippy auto fixes
[x] Made sure my code didn't add any additional warnings: make check
[x] Updated documentation if needed (API.md/README.md/Other)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.