fixed outgoing route to handle links with querystrings in it (using req.query instead of req.url). closes #154.

v1.18.x
Julian Lam 12 years ago
parent e2d0ca9669
commit 08a45e40a2

@ -173,7 +173,7 @@ var RDB = require('./redis.js'),
var href = this.attr('href');
if (href && !href.match(domain)) {
this.attr('href', domain + 'outgoing?' + href);
this.attr('href', domain + 'outgoing?url=' + encodeURIComponent(href));
if (!isSignature) this.append(' <i class="icon-external-link"></i>');
}
});

@ -139,10 +139,11 @@ var user = require('./../user.js'),
});
app.get('/api/outgoing', function(req, res) {
var url = req.url.split('?');
if (url[1]) {
var url = req.query.url;
if (url) {
res.json({
url: url[1],
url: url,
home: global.nconf.get('url')
});
} else {

@ -368,19 +368,15 @@ var express = require('express'),
});
app.get('/outgoing', function(req, res) {
var url = req.url.split('?');
if (url[1]) {
app.build_header({ req: req, res: res }, function(err, header) {
res.send(header + templates['outgoing'].parse({
url: url[1],
home: global.nconf.get('url')
}) + templates['footer']);
});
} else {
res.status(404);
res.redirect(global.nconf.get('relative_path') + '/404');
}
if (!req.query.url) return res.redirect('/404');
app.build_header({ req: req, res: res }, function(err, header) {
res.send(
header +
'\n\t<script>templates.ready(function(){ajaxify.go("outgoing?url=' + req.query.url + '");});</script>' +
templates['footer']
);
});
});
app.get('/search', function(req, res) {

Loading…
Cancel
Save