Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

What happens if you background the widget in mobile browsers? #41

Closed
noahlevenson opened this issue Nov 25, 2022 · 3 comments
Closed

What happens if you background the widget in mobile browsers? #41

noahlevenson opened this issue Nov 25, 2022 · 3 comments

Comments

@noahlevenson
Copy link
Contributor

Does it keep going, or does it halt? The answer to this question probably touches:

#34

https://github.com/getlantern/product/issues/40

https://github.com/getlantern/engineering/issues/78

@noahlevenson
Copy link
Contributor Author

Related: #45

@noahlevenson
Copy link
Contributor Author

noahlevenson commented Dec 2, 2022

Answer: it depends on the platform. It halts on iOS, but it actually continues execution on Android!

Repro:

  1. Start Freddie, the egress server, and a desktop client. Load the widget in mobile Chrome on iOS. Toggle the slider to begin sharing your connection, and let all the connections establish themselves. Proxy some traffic through Firefox as you normally would to poke the system.

  2. Background mobile Chrome by selecting another application. The egress server will report loss of WebSocket connections and the desktop client will report loss of WebRTC connections, indicating that the widget died when you backgrounded it.

  3. Repeat the same steps, but with the widget loaded in mobile Chrome on an Android device. When you background the widget, the connections remain open, and you can continue proxying Firefox through it. (I left the widget backgrounded for ~5 minutes, and it did not cease execution within that span).

Tested on: iPhone 8 running iOS 16.1.2, Android Galaxy Tab A 8.0" running Android 11

@noahlevenson
Copy link
Contributor Author

Closing this because @woodybury merged a backgrounding solution for Android in 96e24de. If I'm misunderstanding things, feel free to reopen it.

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

No branches or pull requests

1 participant