Skip to content

Conversation

@necessarily-equal
Copy link
Contributor

@necessarily-equal necessarily-equal commented Jan 12, 2023

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 Unvanquished/Unvanquished#1926.

@slipher
Copy link
Member

slipher commented Jan 12, 2023

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

@necessarily-equal
Copy link
Contributor Author

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.

@slipher
Copy link
Member

slipher commented Jan 13, 2023

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?

@illwieckz
Copy link
Member

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.

@DolceTriade
Copy link
Contributor

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

@illwieckz
Copy link
Member

@necessarily-equal better rebase than merge, as the merge commit adds useless noise.

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.

drop UDP .dpk transfer

4 participants