Daemon icon indicating copy to clipboard operation
Daemon copied to clipboard

Disable udp download if http download is available

Open necessarily-equal opened this issue 2 years ago • 5 comments

Hey, so we wanted to do something about udp downloads for a while. This is a proposal where UDP downloads will only be proposed if HTTP downloads are not set up.

I'm also fine with nuking the feature though. Downloading things at 15kB/s is just terrible. And according to @cu-kai, if we increase the maximum bandwidth, « but.. if more than one person starts doing this [downloading over udp], the server feels it ». Our UDP protocol is just terrible at downloads. This was apparently done because some people used an old tremulous client at the time, something we don't do.

Note: in the current state this received little to no testing. This can come later once people say if they like the idea or not.

Fixes https://github.com/Unvanquished/Unvanquished/issues/1926.

necessarily-equal avatar Jan 12 '23 20:01 necessarily-equal

How about making a sv_udpDownloads cvar to enable it or not, instead of having it randomly depend on HTTP cvars

slipher avatar Jan 12 '23 21:01 slipher

If we have a cvar for that, then we can't disable udp download as soon as www download is setup. That kind of defeats the point of this: making udp downloads obsolete should there be any other option. I'd rather know if my http download setup is borked that having it fallback immediately to something that is almost unusable.

necessarily-equal avatar Jan 12 '23 22:01 necessarily-equal

If we have a cvar for that, then we can't disable udp download as soon as www download is setup.

What do you mean? You could set the cvar to disable UDP right next to where you set the one for HTTP url

I'd rather know if my http download setup is borked

You mean by seeing people disconnect from the server?

slipher avatar Jan 13 '23 05:01 slipher

To me the UDP download is simply too slow to be usable, meaning we don't use it and we never use it and every tutorial giving instructions to enable download would document http download. Our downloadable content is too large for UDP download.

If we don't know how to make UDP download faster, we better remove it entirely, as it is simply unused and will never be.

To make autodownload standalone, I would favor embedding a minimalistic http server we would run on the same port than UDP port but as TCP port. This would also be reusable for other things like downloading levelshot for map vote screen or things like that.

illwieckz avatar Jan 16 '23 08:01 illwieckz

A minimal HTTP server isn't a bad idea I suppose.

DolceTriade avatar Jan 17 '23 00:01 DolceTriade