Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

chore: nginx optimisations #3423

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

chore: nginx optimisations #3423

Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
02502f1
chore: nginx optimisations
ovflowd Jul 14, 2023
6db48b0
chore: removed duplicated entries and better cache time
ovflowd Jul 14, 2023
b2b8652
chore: limit to 5k conn
ovflowd Jul 14, 2023
ef1dc25
chore: fix wrong line being removed
ovflowd Jul 14, 2023
31089ae
chore: text updates
ovflowd Jul 14, 2023
14d7b49
Update nodejs.org
ovflowd Jul 17, 2023
f705bb2
chore: too many open files
ovflowd Jul 17, 2023
2af00e2
chore: even further reducing limits
ovflowd Jul 17, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
38 changes: 23 additions & 15 deletions ansible/www-standalone/resources/config/nodejs.org
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,10 +23,17 @@ server {
# These directives prevent our server from continuously attempting to open the same requested files on disk
# Open file descriptors and basic metadata for each file requested gets cached
# For specific location blocks, such as /dist, we also cache file not found (404) and other disk errors
open_file_cache max=100000 inactive=300s;
open_file_cache_valid 120s;
open_file_cache_min_uses 2;
open_file_cache_errors off;
open_file_cache max=500 inactive=5m;
open_file_cache_valid 2m;
open_file_cache_min_uses 1;
open_file_cache_errors on;

# This limits the number of connections to the server by creating a maximum amount of concurrent connections
# We limit the number of concurrent connections to be 5000
# This can be fine-tuned; See https://docs.nginx.com/nginx/admin-guide/security-controls/controlling-access-proxied-http/
limit_conn_zone $server_name zone=limitconnserver:10m;
limit_conn_status 429;
limit_conn limitconnserver 5000;

root /home/www/nodejs;
default_type text/plain;
Expand Down Expand Up @@ -62,9 +69,9 @@ server {
# We use ^~ to tell NGINX not to process any other location directive or rewrite after this match
location ^~ /dist {
alias /home/dist/nodejs/release;

autoindex on;
default_type text/plain;
open_file_cache_errors on;

location ~ \.json$ {
add_header access-control-allow-origin *;
Expand Down Expand Up @@ -96,10 +103,17 @@ server {
# These directives prevent our server from continuously attempting to open the same requested files on disk
# Open file descriptors and basic metadata for each file requested gets cached
# For specific location blocks, such as /dist, we also cache file not found (404) and other disk errors
open_file_cache max=100000 inactive=300s;
open_file_cache_valid 120s;
open_file_cache_min_uses 2;
open_file_cache_errors off;
open_file_cache max=500 inactive=5m;
open_file_cache_valid 2m;
open_file_cache_min_uses 1;
open_file_cache_errors on;

# This limits the number of connections to the server by creating a maximum amount of concurrent connections
# We limit the number of concurrent connections to be 5000
# This can be fine-tuned; See https://docs.nginx.com/nginx/admin-guide/security-controls/controlling-access-proxied-http/
limit_conn_zone $server_name zone=limitconnserver:10m;
limit_conn_status 429;
limit_conn limitconnserver 5000;

# We set a default language to "en". This is used for the Localized 404 pages
set $lang en;
Expand Down Expand Up @@ -186,7 +200,6 @@ server {
alias /home/dist/nodejs/release;
autoindex on;
default_type text/plain;
open_file_cache_errors on;

# This rewrite is done to redirect legacy /dist/staging requests to /dist directly
rewrite ^/dist/staging/(.*)$ /dist/$1 permanent;
Expand Down Expand Up @@ -280,11 +293,6 @@ server {
# as set by the @english_fallback location block
# Otherwise, this will fallback to $lang being "en" as defined numerous lines above
location @localized_404 {
# We disable caching of 404 pages as we always want Cloudflare to check if the file now exists
# Some 404s may be caused by the server reaching maximum concurrent file system open() requests
# Disabling cache allows Cloudflare to re-evaluate the same $uri once our server recovers and then properly cache it
add_header Cache-Control "private, no-store, max-age=0" always;

# If this was a rewritten i18n request from @english_fallback, use the localized 404
# If there is no 404 page for that locale, fallback to the English 404
# As a last resort, fallback to NGINX's default 404. This should never happen, and will emit a [crit]
Expand Down