Skip to content

Commit

Permalink
(legacy) missing redirects after post in vpn_ipsec.php
Browse files Browse the repository at this point in the history
  • Loading branch information
AdSchellevis committed Aug 22, 2015
1 parent deb6f79 commit db5faeb
Showing 1 changed file with 9 additions and 0 deletions.
9 changes: 9 additions & 0 deletions src/www/vpn_ipsec.php
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,8 @@
$config['ipsec']['enable'] = !empty($_POST['enable']) ? true : false;
write_config();
vpn_ipsec_configure();
header("Location: vpn_ipsec.php");
exit;
} elseif (isset($_POST['del_x'])) {
/* delete selected p1 entries */
if (isset($_POST['p1entry']) && count($_POST['p1entry'])) {
Expand All @@ -69,6 +71,8 @@
if (write_config()) {
mark_subsystem_dirty('ipsec');
}
header("Location: vpn_ipsec.php");
exit;
}
} elseif (isset($_POST['delp2_x'])) {
/* delete selected p2 entries */
Expand All @@ -79,6 +83,8 @@
if (write_config()) {
mark_subsystem_dirty('ipsec');
}
header("Location: vpn_ipsec.php");
exit;
}
} else {
// move, delete, toggle items by id.
Expand Down Expand Up @@ -217,7 +223,10 @@
mark_subsystem_dirty('ipsec');
}
}
header("Location: vpn_ipsec.php");
exit;
}

}

// form data
Expand Down

2 comments on commit db5faeb

@fichtner
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't these be with a leading slash for consistency?

@AdSchellevis
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

without is consistent with the other legacy pages, maybe we should change that at some point, but for now I will leave it as it is (/was) :)

Please sign in to comment.