<#841 Refactor localhost endpoint port override to...
# flyte-github
a
#841 Refactor localhost endpoint port override to use str.partition Pull request opened by pingsutw Summary • Replaced
str.split(":")
with
str.partition(":")
for extracting host/port when rewriting the localhost console URL from port 8090 to 8080 •
partition()
is safer (no
IndexError
if port is missing) and produces clearer variable names (
host
,
port
) • Updated test expectation to match the port rewrite behavior Test plan •
pytest tests/flyte/remote/test_console.py
— all 11 tests pass flyteorg/flyte-sdk