From 8138917d0bbc71591dac8cb51393bdf1c3383a6b Mon Sep 17 00:00:00 2001
From: MICHAEL JACKSON <mjijackson@gmail.com>
Date: Tue, 23 May 2017 16:06:49 -0700
Subject: [PATCH] Only record referers with hostnames

---
 workers/ingest-logs.js | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/workers/ingest-logs.js b/workers/ingest-logs.js
index b92a4b9..fd78cb2 100644
--- a/workers/ingest-logs.js
+++ b/workers/ingest-logs.js
@@ -1,6 +1,5 @@
 require('isomorphic-fetch')
 const parseURL = require('url').parse
-const formatURL = require('url').format
 const crypto = require('crypto')
 const invariant = require('invariant')
 const admin = require('firebase-admin')
@@ -172,9 +171,9 @@ const computeLogChanges = (stream) =>
 
         // Q: How many requests per day do we receive from a hostname?
         const referer = clientRequest.referer
+        const hostname = referer && parseURL(referer).hostname
 
-        if (referer) {
-          const hostname = parseURL(referer).hostname
+        if (hostname) {
           const key = `requestsPerDayAndRefererHostname/${dayKey}/${hashKey(hostname)}`
 
           if (changes[key]) {