Skip to content

Commit

Permalink
Fix XSS on login page by removing $_REQUEST[path]
Browse files Browse the repository at this point in the history
  • Loading branch information
markjcrane committed Jul 25, 2021
1 parent ffd901b commit c3b8113
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions resources/login.php
Original file line number Diff line number Diff line change
Expand Up @@ -242,11 +242,6 @@
//set variable if not set
if (!isset($_SESSION['login']['domain_name_visible']['boolean'])) { $_SESSION['login']['domain_name_visible']['boolean'] = null; }

//set the requested destination after login
if (!empty($_REQUEST['path'])) {
$_SESSION['login']['destination']['url'] = $_REQUEST['path'];
}

//set a default login destination
if (strlen($_SESSION['login']['destination']['url']) == 0) {
$_SESSION['login']['destination']['url'] = PROJECT_PATH."/core/user_settings/user_dashboard.php";
Expand Down

0 comments on commit c3b8113

Please sign in to comment.