[quake3] HTTP/FTP Downloads (again)

[Shane Isley]

Hi Tony and Co,

I think you and Hobbes have both done a great job on this, it's a 
feature that make a *real* difference.

The one aspect I think needs to change is a server actively disabling 
http downloads. A client should get the missing files as fast as 
possible; it's what the patch does so well.

In my opinion
cl_allowdownload    [integer]
    It should be the clients single download switch.

    0 - don't download
    1 - UDP downloads
    2 - HTTP & UPD downloads

    Alternatively use a 0,1,2,4 bitmask.

cl_dlURL   [string]
    This would serve a similar function to sv_dlURL. It is the fallback 
when the server doesn't supply sv_dlURL (ie the server operator doesn't 
want to or can't be arssed to offer the pak's via http). It allows the 
client to try a trusted URL where the pak's might be; if unavailable the 
client would request to download via UDP. If all three have failed at 
that point, i guess it's time to google or find another server.
   

sv_allowdownloads [integer]
    This should keep it's original role; the servers gatekeeper for 
direct UDP downloads from it process to quake3 clients.

    0 - no UDP downloads
    1 - allow UDP downloads

sv_dlURL   [string]
    This is the place where the server can either actively help clients 
by displaying a URL for it's files. If left blank  "no harm no foul" the 
client can try from it's fall back cvar cl_dlURL.

sv_wwwDlDisconnected    [integer]
    Sounds like this is necessary from your comments on bz.



Excuse the messy layout but I hope you see that we can get more out of 
this top patch.

Regards,
Shane
   


Tony J. White wrote:

>https://bugzilla.icculus.org/show_bug.cgi?id=2661#c20
>
>I've made some cvar related changes to the cURL support patch.
>
>To review, the ET look-alike cvars for this have been dropped in an effort
>to make the impact on SERVERINFO string much smaller.
>
>The new server cvars for controlling this feature are:
>
>sv_allowDownload [integer]
>	This cvar already exists in SERVERINFO and previously the only values
>        recognized were either 0 (off) non-0 (on).  Now this is a bit-flag
>	value with any of the following combinations
>
>	1 - traditional UDP downloading allowed
>	2 - download redirection allowed (cURL)
>	4 - client should disconnect when using download redirection
>
>	This makes the only valid settings:
>	 0 - no downloads
>	 1 - udp downloads only
>	 2 - redirected downloads only
>	 3 - either udp or redirected downloads allowed
>         6 - disconnected redirect downloads only
>	 7 - udp download or disconnected redirect downloads
>
>sv_dlURL [string]
>	This used to be sv_wwwBaseURL in earlier patches.  A string that
>	represents the base of the download URL.  For example, setting 
>	this to "http://tjw.org" may try to download a pak 
>	at "http://tjw.org/baseq3/test.pk3"
>
>I will be testing this latest patch the three main platforms.  If
>all goes well in testing and nobody has any concerns about this patch I
>will commit it to trunk.
>
>-Tony
>  
>