nginx: fix bad tgz storage path

Because of the extra /-/ in the path, nginx was failing to create the
necessary subdirectories to store the file in. Additionally, the parent
folder did not exist, so make sure all the storage paths used are
created at container startup.

Author: rmg

entrypoint.sh

@@ -16,7 +16,9 @@ export MAXAGE=${MAXAGE:-5m}
 # necessary because nginx requires a resolver when upstreams are dynamic
 dnsmasq --listen-address=127.0.0.1 --user=root
 
-# in case it hasn't been created as a tmpfs already
-mkdir -p /tmp/npm
+# in case /tmp/npm hasn't been created as a tmpfs already
+mkdir -p /tmp/npm/{store,temp,cache}
+# nginx runs as uid nobody and it needs write access
+chown -R nobody /tmp/npm
 
 exec /usr/local/openresty/bin/openresty -c /nginx.conf $*

nginx.conf

@@ -37,9 +37,9 @@ http {
         set_by_lua_block $npm_config_registry {
             return ngx.shared.npmConfig:get('npm_config_registry')
         }
-        location ~ ^/.*\.tgz$ {
+        location ~ ^/(.+)/-/(.+\.tgz)$ {
             root               /tmp/npm/store;
-            try_files          $uri @fetch-tgz;
+            try_files          /$1/$2 @fetch-tgz;
         }
         location / {
             set $ephemeralCacheStatus '-';
@@ -70,7 +70,7 @@ http {
             proxy_pass         $npm_config_registry;
             proxy_buffering    on;
             proxy_buffers      512 512k;
-            proxy_store        on;
+            proxy_store        /tmp/npm/store/$1/$2;
             proxy_store_access user:rw group:rw all:r;
             proxy_temp_path    /tmp/npm/temp;
             root               /tmp/npm/store;