From e8919df11047599c9df3791a016a9cd5ea91eb4f Mon Sep 17 00:00:00 2001 From: Jamie Nguyen Date: Mon, 21 Dec 2015 12:13:56 +0000 Subject: [PATCH 1/2] Return 204 instead of 404 for favicon.ico requests --- config/nginx.sample.conf | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/config/nginx.sample.conf b/config/nginx.sample.conf index 930b2c161..7a79b2d78 100644 --- a/config/nginx.sample.conf +++ b/config/nginx.sample.conf @@ -80,11 +80,9 @@ server { internal; } - # return a cheap 404 for favicon cause lots of browsers like asking for it - # view source and old browsers, this bypasses the rails stack at the expense of - # having an ugly 404 page + # bypass rails stack with a cheap 204 for favicon.ico requests location /favicon.ico { - return 404; + return 204; } location / { From eed86e9afcc20b2577bed856e89cc4dc7f5cafda Mon Sep 17 00:00:00 2001 From: Jamie Nguyen Date: Mon, 21 Dec 2015 12:14:36 +0000 Subject: [PATCH 2/2] Do not log favicon.ico requests --- config/nginx.sample.conf | 2 ++ 1 file changed, 2 insertions(+) diff --git a/config/nginx.sample.conf b/config/nginx.sample.conf index 7a79b2d78..897ce6dda 100644 --- a/config/nginx.sample.conf +++ b/config/nginx.sample.conf @@ -83,6 +83,8 @@ server { # bypass rails stack with a cheap 204 for favicon.ico requests location /favicon.ico { return 204; + access_log off; + log_not_found off; } location / {