feat(mobile): apply guest proxy and connectivity - #226
Merged
Merged
Conversation
Add a connectivity feature that stores proxy host and port in Product Store settings, keeps credentials in SecureStore, and routes Discovery and Following fetches through OkHttp when a proxy is on. Expo Go fails closed. Settings and Diagnostics show the live network and proxy state. Closes #162
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Pass Date.now when the runtime clock is omitted so tsc does not treat an explicit undefined as an optional now callback.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #162
Settings now owns a working Proxy panel. Diagnostics shows the same network and proxy state. Discovery and Following fetches honor a saved HTTP proxy through the StreamFusion Development OkHttp module. Expo Go fails closed instead of pretending
fetchused the proxy.Proxy host and port live in Product Store
proxy.v1. Username and password live in SecureStoreproxy.credentials.v1. Native requests cancel by id and retry once on IO failure. Playback playlist classes stay off until Watch exists.expo-networkis pinned to 57.0.1 because 57.0.2 is newer than this environment’s npm cutoff.getNetworkStateAsyncis the online/offline read. The event listener is unused because it can stick at offline after a freeze.createConnectivityRuntimedefaultsnowtoDate.nowwhen the caller omits a clock sotsc --noEmitpasses underexactOptionalPropertyTypes.Merged
mainafter #225. The coverage matrix keeps W11 proxy rows and Channel Detail Guest Follow. Totals: 29 implemented, 14 partial, 12 placeholder, 130 missing.Mobile typecheck is clean. Coverage matrix, connectivity, and channel-follow tests passed. Desktop pre-commit smoke was healthy.