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

Tor not working (neither Price nor Exchange) #1450

Open
ghost opened this issue May 13, 2024 · 3 comments
Open

Tor not working (neither Price nor Exchange) #1450

ghost opened this issue May 13, 2024 · 3 comments

Comments

@ghost
Copy link

ghost commented May 13, 2024

Describe the bug
When set to tor only mode, neither the price works, nor the exchange.

Platform:

  • OS: Android 14
  • Device: Pixel 7 Pro
  • Cake Wallet Version: 4.16.0

Additional context
At first glance it looks like the tor functionality was never even fully tested after #1158

Possibly related to #883 and #1139

@tuxpizza
Copy link
Collaborator

Hi, Cake Wallet currently does not have a built-in Tor daemon, however this feature is coming soon. In order to use the Tor options in Cake, you must run a separate Tor service on your device like Orbot.

@ghost
Copy link
Author

ghost commented May 14, 2024

@tuxpizza I would recommend putting an "alpha" label on this project, given its half-baked nature.

To not warn users that core functionality referenced in the UI is lacking is quite inexcusable for a product that requires this level of trust.

@tuxpizza
Copy link
Collaborator

tuxpizza commented May 14, 2024

Those toggles say "Tor only", meaning it will only attempt to connect over an Onion URL, not that the app would run a Tor service itself. Many other apps rely on an external service to use Tor. These options were added (AFAIK) quite a long time ago and they have only ever been dedicated Onion URLs. With the existing Monero libs, proper app-wide Tor support wouldn't have been very much possible, but with updated Monero libs in the next few weeks it will have a built-in Tor service that will cover the entire app.

Its possible we could explain it better, change the terminology, or add a link to the guide on that Privacy settings page to make it easier to know how to use Tor with Cake (https://guides.cakewallet.com/docs/advanced-features/tor-with-orbot/) but like I mentioned, Cake will have its own Tor service implemented very soon.

To add, that Merged PR you mentioned does not mean that feature is enabled in a production release yet, and as you can see it still has a "Waiting Tests" label. In this case, some preliminary code was merged, but the feature is not enabled/fully working yet.

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