pull/192/head
Michael Q Larson 2015-03-11 23:08:06 -07:00
commit 3004c64ee7
1 changed files with 18 additions and 7 deletions

View File

@ -86,10 +86,12 @@ exports.recent = function(req, res) {
exports.preSubmit = function(req, res) {
var data = req.query;
var cleanData = sanitizeHtml(data.url);
var cleanData = sanitizeHtml(data.url, {
allowedTags: [],
allowedAttributes: []
}).replace(/";/g, '"');
if (data.url.replace(/&/g, '&') !== cleanData) {
debug('data and cleandata', data, cleanData, data.url === cleanData);
req.flash('errors', {
msg: 'The data for this post is malformed'
});
@ -226,7 +228,10 @@ exports.comments = function(req, res, next) {
exports.newStory = function(req, res) {
var url = req.body.data.url;
var cleanURL = sanitizeHtml(url);
var cleanURL = sanitizeHtml(url, {
allowedTags: [],
allowedAttributes: []
}).replace(/"/g, '"');
if (cleanURL !== url) {
req.flash('errors', {
msg: "The URL you submitted doesn't appear valid"
@ -291,10 +296,16 @@ exports.storySubmission = function(req, res) {
link = 'http://' + link;
}
var story = new Story({
headline: sanitizeHtml(data.headline),
headline: sanitizeHtml(data.headline, {
allowedTags: [],
allowedAttributes: []
}).replace(/"/g, '"'),
timePosted: Date.now(),
link: link,
description: sanitizeHtml(data.description),
description: sanitizeHtml(data.description, {
allowedTags: [],
allowedAttributes: []
}).replace(/"/g, '"'),
rank: 1,
upVotes: data.upVotes,
author: data.author,
@ -320,7 +331,7 @@ exports.commentSubmit = function(req, res) {
{
allowedTags: [],
allowedAttributes: []
});
}).replace(/"/g, '"');
if (data.body !== sanitizedBody) {
req.flash('errors', {
msg: 'HTML is not allowed'
@ -346,7 +357,7 @@ exports.commentOnCommentSubmit = function(req, res) {
{
allowedTags: [],
allowedAttributes: []
});
}).replace(/"/g, '"');
if (data.body !== sanitizedBody) {
req.flash('errors', {
msg: 'HTML is not allowed'