Skip to content

Enable TCPing for single-server custom profiles - #6236

Open
eliotcougar wants to merge 3 commits into
2dust:masterfrom
eliotcougar:codex/custom-config-tcping
Open

eliotcougar wants to merge 3 commits into
2dust:masterfrom
eliotcougar:codex/custom-config-tcping

Conversation

@eliotcougar

@eliotcougar eliotcougar commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

Summary

TCPing currently skips every custom profile. Resolve its target from the saved raw JSON so configs with exactly one valid server address and port can be tested. Multiple-server configs and invalid endpoints remain skipped, and stale cached addresses are never used as a fallback.

Depends on #6235. Until that parser fix is merged, this PR also includes the parser changes. The TCPing-only change is limited to the worker and its JVM regression tests.

Related to #6233 (custom-profile TCPing).

Validation

  • 107 JVM tests passed, including 8 new TCPing endpoint cases.
  • Play Store debug and release builds passed; native packaging checked for all ABIs.
  • Emulator UI tests confirmed successful TCP connections for two single-server custom configs and a normal control, with no connections to multiple-server or stale cached endpoints. Stored results and raw JSON were checked afterward.

Not run: physical-device, public-server connectivity, or native real-delay runtime tests. Emulator validation used a local TCP listener with the app's declared local-network permission granted.

@eliotcougar
eliotcougar force-pushed the codex/custom-config-tcping branch from 9af65c4 to 986c663 Compare September 15, 2026 15:58
@eliotcougar
eliotcougar marked this pull request as ready for review September 15, 2026 16:56
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