search

FAForever / kotlin-ice-adapter

MIT License
2 stars 3 forks source link

Add TelemetryClient and implement update coturn server list #14

Closed hieberr closed 6 months ago

hieberr commented 6 months ago

The first in a series of PRs porting over telemetry logic from the java ice adapter.

This PR adds the Telemetry client (really just enables it, most of the code was already there). It also implements the bit of telemetry logic that updates the coturn server list.

Tested using the test telemetry server:

Brutus5000 commented 6 months ago

Apart from the json string looks good, but is more like a dummy. Do you want this to be merged and start on a new PR for further functionality or keep this open?

hieberr commented 6 months ago

Apart from the json string looks good, but is more like a dummy. Do you want this to be merged and start on a new PR for further functionality or keep this open?

I generally prefer smaller PRs merged frequently. So I say merge it unless it complicates your branch (icebreaker-sessions). This one is pretty small though. I would also be happy to leave this open and base future PRs off of it.