From 81bdd2328da34ec48ae6f31ea1e20bd2a45f679a Mon Sep 17 00:00:00 2001
From: Sam Saffron <sam.saffron@gmail.com>
Date: Tue, 17 Nov 2015 19:34:05 +1100
Subject: [PATCH] FIX: stop sending a blank /favicon.ico

instead have nginx ship a 404 for it.
---
 config/nginx.sample.conf |   7 +++++++
 public/favicon.ico       | Bin 198 -> 0 bytes
 2 files changed, 7 insertions(+)
 delete mode 100644 public/favicon.ico

diff --git a/config/nginx.sample.conf b/config/nginx.sample.conf
index 6e846f2a9..e9d815fc9 100644
--- a/config/nginx.sample.conf
+++ b/config/nginx.sample.conf
@@ -80,6 +80,13 @@ 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
+  location /favicon.ico {
+    return 404;
+  }
+
   location / {
     root $public;
     add_header ETag "";
diff --git a/public/favicon.ico b/public/favicon.ico
deleted file mode 100644
index 25dd82c5c1c197f3518ddc2f37381ee353726e28..0000000000000000000000000000000000000000
GIT binary patch
literal 0
HcmV?d00001

literal 198
xcmZQzU<5(|0VW0z*Z{<0K&%180zj+)#2|SPZ~&7K0-_p8fI{Z~{|pS2U;sLCA!h&p