Skip to content

Update and use the Trocador onion address - #1461

Open
sneurlax wants to merge 2 commits into
stagingfrom
feat/trocador-onion
Open

sneurlax wants to merge 2 commits into
stagingfrom
feat/trocador-onion

Conversation

@sneurlax

@sneurlax sneurlax commented Oct 3, 2026

Copy link
Copy Markdown
Member

No description provided.

Trocador retired trocadorfyhlu27aefre5u7zri66gudtzdyelymftvr4yjwcxhfaqsid.onion
and now serves the API from 65bsisadnxvw4kfz7h7a3jwcyenrhluuj3kd5toslfzxbk5q4m3wy6qd.onion
over plain HTTP. Nothing used the constant yet, so this only prepares it.
Every caller hard-coded isOnion: false, so Tor users still exited Tor
to reach api.trocador.app. Decide the authority in one place from the
same condition that selects the SOCKS proxy: when the request goes
through Tor it targets the onion, otherwise the clearnet host.
@codecov

codecov Bot commented Oct 3, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 15 lines in your changes missing coverage. Please review.
✅ Project coverage is 6.79%. Comparing base (1dd116f) to head (a8e46a5).
⚠️ Report is 1 commits behind head on staging.

Files with missing lines Patch % Lines
lib/services/exchange/trocador/trocador_api.dart 0.00% 13 Missing ⚠️
.../services/exchange/trocador/trocador_exchange.dart 0.00% 2 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##           staging    #1461   +/-   ##
========================================
  Coverage     6.79%    6.79%           
========================================
  Files         1122     1122           
  Lines       112368   112367    -1     
========================================
  Hits          7634     7634           
+ Misses      104734   104733    -1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

1 participant