diff --git a/asu/routers/api.py b/asu/routers/api.py index b26c85de..06b604a1 100644 --- a/asu/routers/api.py +++ b/asu/routers/api.py @@ -209,9 +209,6 @@ def api_v1_build_post( request: Request, user_agent: str = Header(None), ): - # Sanitize the profile in case the client did not (bug in older LuCI app). - build_request.profile = build_request.profile.replace(",", "_") - add_build_event("requests") request_hash: str = get_request_hash(build_request) diff --git a/asu/util.py b/asu/util.py index 6daf3fe3..7da271fd 100644 --- a/asu/util.py +++ b/asu/util.py @@ -168,7 +168,7 @@ def get_request_hash(build_request: BuildRequest) -> str: build_request.version, build_request.version_code, build_request.target, - build_request.profile, + build_request.profile.replace(",", "_"), get_packages_hash( build_request.packages_versions.keys() or build_request.packages ), @@ -578,7 +578,7 @@ def reload_profiles(app: FastAPI, version: str, target: str) -> bool: ) app.profiles[version][target] = { - name.replace(",", "_"): profile + name: profile for profile, data in response.json()["profiles"].items() for name in data.get("supported_devices", []) + [profile] } diff --git a/tests/test_api.py b/tests/test_api.py index fb62343c..42f35a35 100644 --- a/tests/test_api.py +++ b/tests/test_api.py @@ -537,7 +537,7 @@ def test_api_build_real_ath79(app): target="ath79/generic", version="23.05.5", packages=["tmux", "vim"], - profile="8dev,carambola2", # Test unsanitized profile. + profile="8dev_carambola2", ), )