Conversation
…21), most recent Tor.framework release without code change
|
I'm getting some tor related errors when closing the app on iOS. They don't seem to persistently break functionality when I reopen the app, but I wanted to bring this to your attention. ===================================================
|
|
I'm seeing that there's now some merge conflicts with the Podfile and one of the Xcode project files. I'll rebase off the latest |
|
@adrastaea @holmesworcester regarding the crash Taea found earlier:
For one thing, the type of crash is especially frustrating because this PR doesn't change how the iOS app integrates with Tor at all (this PR is essentially just the necessary boilerplate needed to get our xcode project building with newer tor releases) My $0.02 is that if and only if we're only seeing these crashes after stopping+reopening Quiet, and if the app never breaks/crashes in a way that negatively impacts a user, then I think i'ts acceptable to merge this PR even though there are technically still crashes being logged that the user is unaware of. Although, with the caveat that we open a ticket to investigate how the biggest If on the other hand Taea's crash persistently breaks functionality then of course this PR is no good in its current state and we'd have to go back to the drawing board and figure out how to safely move to newer versions of tor. In such a scenario, a valuable stopgap strategy can be incrementally upgrading Tor.Framework release-by-release up from the antique But in either case, there are still very important reasons to prioritize getting a working Tor upgrade shipped out ASAP ... As the half-decade chasm between the unsupported tor v It seems like it'd be very wise for us to take a look at how the biggest+most popular Tor.Framework iOS apps respond to system life cycle events (such as app closing/reopening). Since Taea can reproduce this issue around closing+reopening, my gut instinct is that Quiet is most likely improperly integrating with all of the lifecycle events/system states that are to be expected of modern iOS apps. It could be possible that this Tor crash is occurring as a side effect of this larger issue. Or it could be, that that this issue was silently present before this PR - the only difference being that the PR's modern Tor invokes Onion Browser and orbot for iOS seem like the best places to start looking... |
adrastaea
left a comment
There was a problem hiding this comment.
It seemed in my testing like it was safe. I just wanted to check with you that the crash text didn't seem more serious.
Was having difficulty testing the iOS app on the nearly 6 year old version of Tor included in the app. It supported v2 onion services. The logs would be flooded with warnings saying this Tor is old and that it'd have a harder and harder time forming circuits over the tor network.
@kingalg had some problems with Tor 0.4.8.21 when that was included in the big PR. This updates to the newer 0.4.9 series and the changes can be tested in isolation on this branch.