Skip to content

Commit f34d65e

Browse files
authored
verify redirect behind a proxy (#900)
1 parent 68dd126 commit f34d65e

File tree

3 files changed

+6
-10
lines changed

3 files changed

+6
-10
lines changed

public_html/lists/admin/js/phplistapp.js

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -304,15 +304,11 @@ $(document).ready(function () {
304304
}
305305
});
306306

307-
308-
309307
$("a.savechanges").on("click",function () {
310-
if (changed) {
311-
document.sendmessageform.followupto.value = this.href;
312-
document.location.hash = ""
313-
document.sendmessageform.submit();
314-
return false;
315-
}
308+
$('#followupto').val(this.href);
309+
document.location.hash = ""
310+
$('#sendmessageform').submit();
311+
return false;
316312
});
317313

318314
$("#criteriaSelect").on("change",function () {

public_html/lists/admin/lib.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1150,7 +1150,7 @@ function fetchStyles($text) {
11501150
function isValidRedirect($url)
11511151
{
11521152
//# we might want to add some more checks here
1153-
return strpos($url, hostName());
1153+
return stripos($url, hostName()) || stripos($url,getConfig('website'));
11541154
}
11551155

11561156
/* check the url_append config and expand the url with it

public_html/lists/admin/send_core.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -607,7 +607,7 @@
607607

608608
// print $tabs->display();
609609
}
610-
echo '<input type="hidden" name="followupto" value="" />';
610+
echo '<input id="followupto" type="hidden" name="followupto" value="" />';
611611

612612
if ($_GET['page'] == 'preparemessage') {
613613
echo Help('preparemessage', $GLOBALS['I18N']->get('What is prepare a message'));

0 commit comments

Comments
 (0)