Skip to content

frontend: disable option to connect via tor proxy on ios #3452

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

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

Conversation

thisconnect
Copy link
Collaborator

The Tor proxy can currently not be used on iOS. iOS restricts exposing local ports to other apps.

Before asking for reviews, here is a check list of the most common things you might need to consider:

  • updating the Changelog
  • writing unit tests
  • checking if your changes affect other coins or tokens in unintended ways
  • testing on multiple environments (Qt, Android, ...)
  • having an AI review your changes

Copy link
Contributor

@benma benma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We also need to disable it in the backend - users who already enabled it in the current iOS release would be stuck after this with no way of disabling it.

CHANGELOG.md Outdated
@@ -1,6 +1,7 @@
# Changelog

## Unreleased
- Disabled the option to enable Tor proxy on iOS
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please re-target this pr to the release-v4.48.1 branch (which I just created) and put this changelog into v4.48.1, as it is fix to the v4.48.1 release.

The Tor proxy can currently not be used on iOS. iOS restricts
exposing local ports to other apps.
@thisconnect thisconnect force-pushed the frontend-disable-tor-on-ios branch from bc3da1c to 6da7ea5 Compare July 17, 2025 05:49
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.

2 participants