mirror of
https://github.com/gesellix/Bose-SoundTouch.git
synced 2026-08-19 09:06:14 +00:00
Tobias GesellchenandClaude Sonnet 5
eec57cbc10
validateURL() unconditionally rejected the hostnames "localhost" and "127.0.0.1" for the four Migrate-tab plan URL fields, with no awareness of deployment mode. Since the Suggested Plan's URLs are derived from the page's own configured Target URL, a fresh on-device install (whose server_url is now correctly http://localhost:8000, since #546) loaded the Migrate tab with "Apply Suggested Plan" and "Pre-flight" disabled by default, before the user touched anything -- directly contradicting the on-device docs' "Migrate -> accept the suggested plan -> apply" instructions. Found while investigating why a #614 reporter used the non-standard "localhost.localdomain" as a workaround, and why a #621 reporter got stuck with "Migration Status: Migrated (URL mismatch)" trying to follow the (correct) on-device localhost guidance. Fix: a loopback URL is only flagged when it doesn't match the plan's own Target URL origin. A field that's exactly what the service itself is already configured to answer as (the on-device case) is accepted; a stray "localhost" typed into one field while Target URL is a real LAN address (the external-host mistake the check exists to catch) is still flagged, since the origins differ. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>