mirror of
https://github.com/TencentCloud/Octop.git
synced 2026-10-20 06:19:17 +08:00
IronMurphy
c4685ba54d
The stream router's _normalize_nav_url() checked t.startswith(("http://", "https://")) with a raw string, so a user-typed HTTPS://host URL was prefixed again into https://HTTPS://host, which the browser then cannot load. Match octop.infra.utils.url.normalize_nav_url's case-insensitive check, and cover the behaviour with tests.
27 lines
929 B
Python
27 lines
929 B
Python
"""tests/unit/api/test_browser_stream_normalize_nav.py"""
|
|
|
|
from __future__ import annotations
|
|
|
|
from octop.api.routers.browser.stream import _normalize_nav_url
|
|
|
|
|
|
def test_empty_and_blank() -> None:
|
|
assert _normalize_nav_url("") == ""
|
|
assert _normalize_nav_url(" ") == ""
|
|
|
|
|
|
def test_bare_host_is_prefixed() -> None:
|
|
assert _normalize_nav_url("baidu.com") == "https://baidu.com"
|
|
assert _normalize_nav_url("example.com/path?q=1") == "https://example.com/path?q=1"
|
|
|
|
|
|
def test_http_schemes_are_preserved() -> None:
|
|
assert _normalize_nav_url("http://x.com/") == "http://x.com/"
|
|
assert _normalize_nav_url("https://x.com/") == "https://x.com/"
|
|
|
|
|
|
def test_uppercase_scheme_is_not_double_prefixed() -> None:
|
|
"""A user-typed HTTPS:// URL must not become https://HTTPS://..."""
|
|
assert _normalize_nav_url("HTTPS://x.com/") == "HTTPS://x.com/"
|
|
assert _normalize_nav_url("HTTP://x.com/") == "HTTP://x.com/"
|