Issue 402 - Use a redirector script for downloads, not a direct link

modules/updateserver/files/site.conf

Index: modules/updateserver/files/site.conf
===================================================================
--- a/modules/updateserver/files/site.conf
+++ b/modules/updateserver/files/site.conf
@@ -1,2 +1,12 @@
 keepalive_timeout 0;
 root /var/www/update;
+
+location /

[mathias, 2014/09/15 16:15:56]

Why is the explicit empty location required?

[Wladimir Palant, 2014/09/15 16:46:47]

That's a very good question. In fact, the official documentation doesn't really
say what happens if a request doesn't match any location whatsoever.
https://www.linode.com/docs/websites/nginx/basic-nginx-configuration#location...
is a bit more explicit but still only recommends: "Make sure each file and
folder under a domain will match at least one location directive." That's
exactly what I did, I didn't bother checking what happens if I leave that
location directive away.

Now that I actually tested this, nginx seems to always generate an implicit
|location / {}| - so removing it here doesn't seem to have any effect. However,
I think that having it here makes things more obvious. Besides, I don't like
relying on undocumented behavior. What do you think?

[mathias, 2014/09/15 17:26:05]

Agreed, leave it as it is. But thank you for the clarification, though ;-)

+{
+}
+
+location /latest/
+{
+  fastcgi_pass unix:/tmp/multiplexer-fastcgi.sock;
+  include /etc/nginx/fastcgi_params;
+}