404 on unknown thumbnail
This commit is contained in:
parent
a8981e0b24
commit
a8bf1d826e
|
@ -48,13 +48,13 @@ staticRouter.use(
|
||||||
const thumbnailsPhysicalPath = CONFIG.STORAGE.THUMBNAILS_DIR
|
const thumbnailsPhysicalPath = CONFIG.STORAGE.THUMBNAILS_DIR
|
||||||
staticRouter.use(
|
staticRouter.use(
|
||||||
STATIC_PATHS.THUMBNAILS,
|
STATIC_PATHS.THUMBNAILS,
|
||||||
express.static(thumbnailsPhysicalPath, { maxAge: STATIC_MAX_AGE })
|
express.static(thumbnailsPhysicalPath, { maxAge: STATIC_MAX_AGE, fallthrough: false }) // 404 if the file does not exist
|
||||||
)
|
)
|
||||||
|
|
||||||
const avatarsPhysicalPath = CONFIG.STORAGE.AVATARS_DIR
|
const avatarsPhysicalPath = CONFIG.STORAGE.AVATARS_DIR
|
||||||
staticRouter.use(
|
staticRouter.use(
|
||||||
STATIC_PATHS.AVATARS,
|
STATIC_PATHS.AVATARS,
|
||||||
express.static(avatarsPhysicalPath, { maxAge: STATIC_MAX_AGE })
|
express.static(avatarsPhysicalPath, { maxAge: STATIC_MAX_AGE, fallthrough: false }) // 404 if the file does not exist
|
||||||
)
|
)
|
||||||
|
|
||||||
// We don't have video previews, fetch them from the origin instance
|
// We don't have video previews, fetch them from the origin instance
|
||||||
|
|
|
@ -53,7 +53,7 @@ server {
|
||||||
alias /var/www/peertube/peertube-latest/client/dist/$1;
|
alias /var/www/peertube/peertube-latest/client/dist/$1;
|
||||||
}
|
}
|
||||||
|
|
||||||
location ~ ^/static/(thumbnails|avatars)/(.*)$ {
|
location ~ ^/static/(thumbnails|avatars)/ {
|
||||||
if ($request_method = 'OPTIONS') {
|
if ($request_method = 'OPTIONS') {
|
||||||
add_header 'Access-Control-Allow-Origin' '*';
|
add_header 'Access-Control-Allow-Origin' '*';
|
||||||
add_header 'Access-Control-Allow-Methods' 'GET, OPTIONS';
|
add_header 'Access-Control-Allow-Methods' 'GET, OPTIONS';
|
||||||
|
@ -64,16 +64,17 @@ server {
|
||||||
return 204;
|
return 204;
|
||||||
}
|
}
|
||||||
|
|
||||||
if ($request_method = 'GET') {
|
add_header 'Access-Control-Allow-Origin' '*';
|
||||||
add_header 'Access-Control-Allow-Origin' '*';
|
add_header 'Access-Control-Allow-Methods' 'GET, OPTIONS';
|
||||||
add_header 'Access-Control-Allow-Methods' 'GET, OPTIONS';
|
add_header 'Access-Control-Allow-Headers' 'Range,DNT,X-CustomHeader,Keep-Alive,User-Agent,X-Requested-With,If-Modified-Since,Cache-Control,Content-Type';
|
||||||
add_header 'Access-Control-Allow-Headers' 'Range,DNT,X-CustomHeader,Keep-Alive,User-Agent,X-Requested-With,If-Modified-Since,Cache-Control,Content-Type';
|
|
||||||
}
|
|
||||||
|
|
||||||
# Cache 2 hours
|
# Cache 2 hours
|
||||||
add_header Cache-Control "public, max-age=7200";
|
add_header Cache-Control "public, max-age=7200";
|
||||||
|
|
||||||
alias /var/www/peertube/storage/$1/$2;
|
root /var/www/peertube/storage;
|
||||||
|
|
||||||
|
rewrite ^/static/(thumbnails|avatars)/(.*)$ /$1/$2 break;
|
||||||
|
try_files $uri /;
|
||||||
}
|
}
|
||||||
|
|
||||||
location / {
|
location / {
|
||||||
|
|
Loading…
Reference in New Issue