Skip to content

SRTLA: choose connection used to register the group by probing RTT#173

Merged
eerimoq merged 1 commit intoeerimoq:mainfrom
rationalsa:srtla_probe
Oct 9, 2025
Merged

SRTLA: choose connection used to register the group by probing RTT#173
eerimoq merged 1 commit intoeerimoq:mainfrom
rationalsa:srtla_probe

Conversation

@rationalsa
Copy link
Copy Markdown
Contributor

This allows the client to find a connection that's 1) working and 2) has the lowest RTT, and only then use it to attempt to register the SRTLA connection group

Should improve connection reliability and speed when some (but not all) of the networks are congested or otherwise have high latency. This is essentially the same functionality introduced in srtla_send in 2023 here:

BELABOX/srtla@f0edf2e...f138fb4

Additionally, we use this to allow clients with connections routed to multiple POPs of our BELABOX cloud anycast network to steer the connection towards a preferred location

@rationalsa
Copy link
Copy Markdown
Contributor Author

rationalsa commented Oct 9, 2025

Oops, I've missed one thing in the state machine there. Will push a fix shortly

@eerimoq
Copy link
Copy Markdown
Owner

eerimoq commented Oct 9, 2025

Run brew install swiftlint swiftformat && make style lint to style the code and do some static analysis to make the test pass. I can do it if you cannot do it.

This allows the client to find a connection that's 1) working and 2)
has the lowest RTT, and only then use it to attempt to register the
SRTLA connection group

Should improve connection reliability and speed when some (but not
all) of the networks are congested or otherwise have high latency.
This is essentially the same functionality introduced in srtla_send
in 2023 here:

BELABOX/srtla@f0edf2e...f138fb4

Additionally, we use this to allow clients with connections routed
to multiple POPs of our BELABOX cloud anycast network to steer the
connection towards a preferred location
@eerimoq eerimoq merged commit fd0f1ad into eerimoq:main Oct 9, 2025
1 check passed
datagutt added a commit to irlserver/srtla_send that referenced this pull request Oct 9, 2025
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