Depends on LOCALSTACK_HOST migration: #9390

Motivation

EDGE_PORT, EDGE_PORT_HTTP and EDGE_BIND_HOST are replaced by GATEWAY_LISTEN, and have been deprecated since version 2.0. These variables should be removed in a major version, and can be done in version 3.0.

Changes

• Make EDGE_PORT read-only. Internal use is widespread and therefore we consider it as a shortcut for now with the semantics EDGE_PORT == GATEWAY_LISTEN[0].port. We also keep exposing EDGE_PORT in LocalStack compute services because we have not advertised a migration path yet.
• Remove EDGE_PORT_HTTP
• Remove EDGE_BIND_HOST

Background

• Announced networking changes upon 2.0: https://discuss.localstack.cloud/t/upcoming-changes-for-localstack-v2/239#networking-7

• GATEWAY_LISTEN behavior regarding protocols http vs https:

  When we bind to a ip/port, we accept both http and https requests, so the protocol distinction is not important for the purposes of our hypercorn server. For example you can send HTTPS requests to both 4566 and 443 in your example above. I think that means that we allow all combinations of port and protocol

• More details in internal decision document: https://www.notion.so/localstack/Migration-path-for-LOCALSTACK_HOSTNAME-and-EDGE_PORT-9cf800595f854add96ce2788a6d93367?pvs=4

Discussion

• CORS is now more permissive (see localstack/aws/handlers/cors.py) and the tests required updates (tests/unit/test_cors.py). Is this ok?
• Monkeypatching of GATEWAY_LISTEN needs to use typed [HostAndPort] rather than the user-facing string-based API. Does this work properly without side effects? Do we need to provide any guards when used with strings?
• StepFunctions V1: Does anything depend on the environment variable EDGE_PORT_HTTP (generated in localstack.services.stepfunctions.stepfunctions_starter.StepFunctionsServer.generate_env_vars) used when starting the Java StepFunctionsLocal.jar ? (I would leave it for now as this StepFunctions implementation will be deprecated anyway).

TODO

• Rebase upon the dependent LOCALSTACK_HOST migration Migrate endpoints to use LOCALSTACK_HOST only #9390
• Fix tests.unit.test_config.TestEdgeVariablesDerivedCorrectly.test_localstack_host_overrides_edge_variables depending on the migration to LOCALSTACK_HOST https://github.com/localstack/localstack/pull/9390/files#diff-8ee48b2bcad465473db1a1b202781ca2c7c23db7bd1670e79d45668fb2bd718fR616
• Remove duplicated commit from Viren from this PR (upon next rebase)
• Synchronize merge with Lambda removal PRs (might have to resolve a few conflicts) Remove legacy lambda provider #9543

Follow-up

• Remove get_edge_port_http() (this refactoring can also be done post v3 to limit the impact; used 13x in LS + 13x in ext)