fix(badbits): return 410 instead of 403

filecoin-saturn / L1-node

Filecoin Saturn L1 Node • The edge cache layer of Filecoin's decentralized CDN 🪐

Other

140 stars 49 forks source link

fix(badbits): return 410 instead of 403 #486

Closed DiegoRBaquero closed 10 months ago

DiegoRBaquero commented 10 months ago

Changes 403s to 410s
Removes auth_request usage and its module
Improves Dockerfile orgnization
Cleans up nginx built modules
Uses new NodeSource Node.js deb usage

Closes #485

guanzo commented 10 months ago

Is there a reason why some variables are declared twice? LASSIE_VERSION, NGINX_VERSION, NGX_BROTLI_COMMIT, NJS_VERSION, NODEJS_MAJOR_VERSION

DiegoRBaquero commented 10 months ago

Is there a reason why some variables are declared twice? LASSIE_VERSION, NGINX_VERSION, NODEJS_MAJOR_VERSION

That's because there's a top-level ARG (for the Dockerfile) and ARG per FROM for the aliased builds. So the top-level and the first seem duplicated, but they are not.