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

Enable Tor #1193

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Enable Tor #1193

wants to merge 1 commit into from

Conversation

drwdal
Copy link

@drwdal drwdal commented Mar 8, 2020

You should allow Tor users to have the choice for anti censorship purposes.

You should allow Tor users to have the choice for anti censorship purposes.
@licaon-kter
Copy link
Contributor

@drwdal
Copy link
Author

drwdal commented Mar 9, 2020

You did read https://nitter.net/ChatSecure/status/1226639098040877057 right?

Yes, as well as the issue by @irlcatgirl who I agree with, you should retain it in the main build for the sake of people who do not have iOS build environments and need to access XMPP over Tor

@drwdal
Copy link
Author

drwdal commented Mar 9, 2020

the benefit of Tor is that a user in a restrictive environment may still be allowed to access a mobile app store, eliminating support for Tor suddenly while said support remains viable puts those users at inconvenience if not at risk for a failure to find an alternative.

@chrisballinger
Copy link
Member

It may be re-enabled at a future date, but it will require additional time and resources to ensure that it is functional.

@elaine-jackson
Copy link

elaine-jackson commented Mar 10, 2020 via email

@chrisballinger
Copy link
Member

@irlcatgirl Lol that's not how any of this works. Fortunately the government still can't compel speech, including forcing companies to implement crypto backdoors.

The simple reality is that there aren't enough paying end users / recurring donations to allow me to work on this software full time. In order to support Mac Catalyst I had to do a refactoring of the Tor control socket code in CPAProxy, and was not able to test it sufficiently before the 5.0 release, so I have it disabled for now.

Want to see more active maintenance? Consider financially supporting the project and encouraging others to do the same.

@drwdal
Copy link
Author

drwdal commented Mar 12, 2020

@irlcatgirl Lol that's not how any of this works. Fortunately the government still can't compel speech, including forcing companies to implement crypto backdoors.

The simple reality is that there aren't enough paying end users / recurring donations to allow me to work on this software full time. In order to support Mac Catalyst I had to do a refactoring of the Tor control socket code in CPAProxy, and was not able to test it sufficiently before the 5.0 release, so I have it disabled for now.

Want to see more active maintenance? Consider financially supporting the project and encouraging others to do the same.

Why not take the point of two interested parties who need this feature and would gladly use it if you put it in the main build, you can just add a disclaimer about reliability, socket control isn't a major issue presuming it's functional at any level, and performance isn't a great reason to exclude it.

@chrisballinger Apple can still limit your exposure on international stores in response to certain features being added/removed, it's easier to assume that would be the first reason you might be encouraged to remove support for Tor.

Want to see more active maintenance? Consider financially supporting the project and encouraging others to do the same.

How much would it realistically cost for you to leave Tor support in the main build for people who don't want the privacy implications of TestFlight? It seems like it would be a minimal effort.

@chrisballinger
Copy link
Member

@drwdal The control socket not working is an issue because Tor support is not functional at all without it. The app is already not available in China due to Tor support, but that happened years ago during the great VPN ban wave.

Have you verified you're able to connect to Tor accounts and .onion services using your patch?

@elaine-jackson
Copy link

elaine-jackson commented Mar 12, 2020 via email

@chrisballinger
Copy link
Member

@irlcatgirl What? It's already not in the Chinese App Store, and no, I wouldn't remove OMEMO.

@elaine-jackson
Copy link

elaine-jackson commented Mar 13, 2020 via email

@licaon-kter
Copy link
Contributor

licaon-kter commented Mar 13, 2020

Monal and Siskin are on the China store with OMEMO, right? https://applecensorship.com/

@chrisballinger
Copy link
Member

@licaon-kter Yep, currently OMEMO is fine, but Tor is not. However any app can be banned at any time, for any reason, within their territory.

@mimi89999
Copy link
Contributor

@irlcatgirl Maybe the government sent you to break ChatSecure so that users switch to less secure alternatives?

@chrisballinger Can this PR be closed unless somebody proves that it's actually working?

@elaine-jackson
Copy link

@irlcatgirl Maybe the government sent you to break ChatSecure so that users switch to less secure alternatives?

@chrisballinger Can this PR be closed unless somebody proves that it's actually working?

Maybe I'm not actually a college student and work for the CIA 😜

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.

5 participants