remove notLocalHost

server/config.js

@@ -3,7 +3,7 @@ const convict = require('convict');
 const conf = convict({
   s3_bucket: {
     format: String,
-    default: 'localhost',
+    default: '',
     env: 'P2P_S3_BUCKET'
   },
   redis_host: {
@@ -19,18 +19,17 @@ const conf = convict({
   },
   analytics_id: {
     format: String,
-    default: 'UA-101393094-1',
+    default: '',
     env: 'GOOGLE_ANALYTICS_ID'
   },
   sentry_id: {
     format: String,
-    default:
-      'https://cdf9a4f43a584f759586af8ceb2194f2@sentry.prod.mozaws.net/238',
+    default: '',
     env: 'P2P_SENTRY_CLIENT'
   },
   sentry_dsn: {
     format: String,
-    default: 'localhost',
+    default: '',
     env: 'P2P_SENTRY_DSN'
   },
   env: {
@@ -45,8 +44,3 @@ conf.validate({ allowed: 'strict' });
 
 const props = conf.getProperties();
 module.exports = props;
-
-module.exports.notLocalHost =
-  props.env === 'production' &&
-  props.s3_bucket !== 'localhost' &&
-  props.sentry_dsn !== 'localhost';

server/log.js

@@ -1,12 +1,12 @@
 const conf = require('./config.js');
 
-const notLocalHost = conf.notLocalHost;
+const isProduction = conf.env === 'production'
 
 const mozlog = require('mozlog')({
   app: 'FirefoxFileshare',
-  level: notLocalHost ? 'INFO' : 'verbose',
-  fmt: notLocalHost ? 'heka' : 'pretty',
-  debug: !notLocalHost
+  level: isProduction ? 'INFO' : 'verbose',
+  fmt: isProduction ? 'heka' : 'pretty',
+  debug: !isProduction
 });
 
 module.exports = mozlog;

server/portal_server.js

@@ -9,9 +9,7 @@ const conf = require('./config.js');
 const storage = require('./storage.js');
 const Raven = require('raven');
 
-const notLocalHost = conf.notLocalHost;
-
-if (notLocalHost) {
+if (conf.sentry_dsn) {
   Raven.config(conf.sentry_dsn).install();
 }
 
@@ -39,7 +37,6 @@ app.use(express.static(STATIC_PATH));
 
 app.get('/', (req, res) => {
   res.render('index', {
-    shouldRenderAnalytics: notLocalHost,
     trackerId: conf.analytics_id,
     dsn: conf.sentry_id
   });
@@ -64,7 +61,6 @@ app.get('/download/:id', (req, res) => {
         res.render('download', {
           filename: filename,
           filesize: bytes(contentLength),
-          shouldRenderAnalytics: notLocalHost,
           trackerId: conf.analytics_id,
           dsn: conf.sentry_id
         });
@@ -93,7 +89,7 @@ app.get('/assets/download/:id', (req, res) => {
         });
         const file_stream = storage.get(id);
 
-        file_stream.on(notLocalHost ? 'finish' : 'close', () => {
+        file_stream.on('end', () => {
           storage
             .forceDelete(id)
             .then(err => {
@@ -149,7 +145,7 @@ app.post('/upload/:id', (req, res, next) => {
   req.busboy.on('file', (fieldname, file, filename) => {
     log.info('Uploading:', req.params.id);
 
-    const protocol = notLocalHost ? 'https' : req.protocol;
+    const protocol = conf.env === 'production' ? 'https' : req.protocol;
     const url = `${protocol}://${req.get('host')}/download/${req.params.id}/`;
 
     storage.set(req.params.id, file, filename, url).then(linkAndID => {

server/storage.js

@@ -6,8 +6,6 @@ const fs = require('fs');
 const path = require('path');
 const crypto = require('crypto');
 
-const notLocalHost = conf.notLocalHost;
-
 const mozlog = require('./log.js');
 
 const log = mozlog('portal.storage');
@@ -22,7 +20,7 @@ redis_client.on('error', err => {
   log.info('Redis:', err);
 });
 
-if (notLocalHost) {
+if (conf.s3_bucket) {
   module.exports = {
     filename: filename,
     exists: exists,