Skip to content
This repository has been archived by the owner on Aug 19, 2024. It is now read-only.

Various fixes #20

Merged
merged 2 commits into from
Apr 19, 2024
Merged

Various fixes #20

merged 2 commits into from
Apr 19, 2024

Conversation

qstokkink
Copy link
Owner

@qstokkink qstokkink commented Apr 19, 2024

Fixes #21

This PR:

  • Fixes a crash when linking the core to the GUI when requests are already destructed.
  • Updates the default minimum circuits to be 6 instead of 1. This should hopefully improve out-of-the-box anon download speeds.

@qstokkink qstokkink marked this pull request as ready for review April 19, 2024 11:59
@egbertbouman
Copy link
Collaborator

With circuits now being able to do 12+MB/s, I don't think you need 6 circuits to achieve decent speeds. Considering exit nodes allow only a limited amount of circuits, I think we should consider a default of 3, like we do on the main repo.

@qstokkink qstokkink changed the title Set the minimum circuits to 6 Various fixes Apr 19, 2024
@qstokkink
Copy link
Owner Author

@egbertbouman 👍 fixed.

Copy link
Collaborator

@egbertbouman egbertbouman left a comment

Choose a reason for hiding this comment

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

Thanks!

@qstokkink qstokkink merged commit 5862b67 into highlander Apr 19, 2024
4 checks passed
@qstokkink qstokkink deleted the upd_min_circuits branch April 19, 2024 12:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Initial connection failing (Ubuntu)
2 participants