Skip to content

Allow separate OpenSpeedTest save and results URLs - #1092

Open
Jason-Morcos wants to merge 1 commit into
Ozark-Connect:mainfrom
Moballo-LLC:agent/configurable-openspeedtest-urls
Open

Allow separate OpenSpeedTest save and results URLs#1092
Jason-Morcos wants to merge 1 commit into
Ozark-Connect:mainfrom
Moballo-LLC:agent/configurable-openspeedtest-urls

Conversation

@Jason-Morcos

Copy link
Copy Markdown
Contributor

Problem

The native OpenSpeedTest template assumes that result submissions and the Network Optimizer results page share one origin derived from SAVE_DATA_URL. Reverse-proxy deployments can legitimately serve the test UI on one host, submit results through a same-origin API route there, and show saved results on the Network Optimizer host.

Today those deployments have to rewrite the installed template after every update.

Fix

  • add optional OPENSPEEDTEST_SAVE_DATA_URL and OPENSPEEDTEST_CLIENT_RESULTS_URL settings
  • preserve the current derived URLs when neither setting is present
  • accept only absolute HTTP(S) URLs or root-relative paths
  • JSON-encode generated JavaScript values rather than interpolating raw strings
  • document the two settings for native macOS installs

Validation

  • tests cover independent routes, unchanged defaults, and rejection of unsafe URL shapes
  • focused tests: 5 passed
  • full Release NetworkOptimizer.Web.Tests: 1,285 passed

No deployed OpenSpeedTest files or proxy routes were changed while validating this fix.

@Jason-Morcos
Jason-Morcos force-pushed the agent/configurable-openspeedtest-urls branch from 3fd61e6 to 41d9532 Compare August 3, 2026 04:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant