Opened 5 weeks ago

Last modified 10 days ago

#31200 merge_ready defect

Hand out proxy-go snowflakes more frequently than webextension snowflakes

Reported by: cohosh Owned by: cohosh
Priority: Medium Milestone:
Component: Circumvention/Snowflake Version:
Severity: Normal Keywords:
Cc: arlolra, cohosh, phw, dcf Actual Points:
Parent ID: #25681 Points:
Reviewer: Sponsor: Sponsor28

Description

We're now seeing several volunteers running both proxy-go instances and webextension Snowflake proxies (yay!). However, throughout the course of my dogfooding, it seems the quality of the connections has degraded and I get frequent disconnects. This requires more investigation, but I suspect the higher ratio of web extension proxies is part of the reason.

Right now we can tell the difference between the two because of the length of the ids. I propose prioritizing the handout of proxy-go instances in the short term until the root of this problem is more thoroughly investigated.

Child Tickets

Change History (8)

comment:1 Changed 5 weeks ago by cohosh

Parent ID: #25681

comment:2 Changed 5 weeks ago by dcf

A cheap and easy way to do this is to increase defaultBrokerPollInterval in config.js, while letting proxy-go continue to use a shorter interval. Related to #25598.

comment:3 Changed 4 weeks ago by cypherpunks

-

Last edited 4 weeks ago by cypherpunks (previous) (diff)

comment:4 Changed 3 weeks ago by cypherpunks

Why is running an instance with proxy-go much faster than running it with the WebExt?

comment:5 Changed 12 days ago by cohosh

Owner: set to cohosh
Status: newassigned

Working on this now to alleviate some reported quality issues with snowflake connections (#31385). The feature suggested in #25598 sounds great, but for now I want to just temporarily increase the polling period for the webextension proxies.

comment:6 Changed 12 days ago by cohosh

Status: assignedneeds_review

https://github.com/cohosh/snowflake/compare/ticket31200

This raises it from every 5 seconds to every 20 seconds.

comment:7 Changed 12 days ago by arlolra

Status: needs_reviewmerge_ready

Sure

comment:8 Changed 10 days ago by cohosh

Merged in f94ef87c46

Note: See TracTickets for help on using tickets.