minor random post-merge fixes

This commit is contained in:
Michael Q Larson
2014-12-23 13:30:20 -08:00
parent ef2eba7b14
commit b4be1f859f
4 changed files with 5 additions and 12 deletions

11
app.js
View File

@ -97,7 +97,7 @@ app.use(helmet.xssFilter());
app.use(helmet.xframe()); app.use(helmet.xframe());
var trusted = [ var trusted = [
'"self"', "'self'",
'*.freecodecamp.com', '*.freecodecamp.com',
'*.gstatic.com', '*.gstatic.com',
"*.google-analytics.com", "*.google-analytics.com",
@ -113,15 +113,6 @@ var trusted = [
"*.rafflecopter.com", "*.rafflecopter.com",
"localhost:3001" "localhost:3001"
]; ];
//TODO(Berks): conditionally add localhost domains to csp;
/*var connectSrc;
if (process.env.NODE_ENV === 'development') {
debug('Pushing');
connectSrc = [''self'', 'ws://localhost:3001/'];
} else {
debug('Not');
connectSrc = [];
}*/
debug(trusted); debug(trusted);
app.use(helmet.contentSecurityPolicy({ app.use(helmet.contentSecurityPolicy({

View File

@ -34,7 +34,7 @@ exports.returnChallenge = function(req, res, next) {
"Prove to mom that computers aren't just for games" "Prove to mom that computers aren't just for games"
]; ];
if (challengeNumber > 59) { challengeNumber = 0; } if (challengeNumber > 59) { challengeNumber = 0; }
Challenge.find({}, function (err, c) { Challenge.find({}, null, { sort: { challengeNumber: 1 } }, function(err, c) {
if (err) { if (err) {
debug('Challenge err: ', err); debug('Challenge err: ', err);
next(err); next(err);
@ -52,4 +52,5 @@ exports.returnChallenge = function(req, res, next) {
challenges: c challenges: c
}); });
}); });
}; };

View File

@ -134,7 +134,7 @@ exports.postEmailSignup = function(req, res, next) {
*/ */
exports.getAccount = function(req, res) { exports.getAccount = function(req, res) {
Challenge.find({}, function (err, c) { Challenge.find({}, null, { sort: { challengeNumber: 1 } }, function(err, c) {
if (err) { if (err) {
console.error('Challenge err: ', err); console.error('Challenge err: ', err);
next(err); next(err);

View File

@ -1,3 +1,4 @@
require('dotenv').load();
var Challenge = require('../models/Challenge.js'), var Challenge = require('../models/Challenge.js'),
mongoose = require('mongoose'), mongoose = require('mongoose'),
secrets = require('../config/secrets'), secrets = require('../config/secrets'),