Ajax headers for miscellaneous forms #3435

Merged
merged 5 commits into from Apr 29, 2016

Projects

None yet

3 participants

@pblasquez
Contributor

Please see #3113 for context, this is the logical grouping for the miscellaneous remaining forms.

pblasquez added some commits Apr 22, 2016
@pblasquez pblasquez Merge remote-tracking branch 'refs/remotes/librenms/master' 4510811
@pblasquez pblasquez Merge remote-tracking branch 'refs/remotes/librenms/master' add7aeb
@pblasquez pblasquez Ajax headers for various
826cd17
@laf laf commented on an outdated diff Apr 28, 2016
html/includes/forms/parse-template-rules.inc.php
@@ -11,6 +11,7 @@
* option) any later version. Please see LICENSE.txt at the top level of
* the source code distribution for details.
*/
+header('Content-type: application/json');
if (is_admin() === false) {
die('ERROR: You need to be admin');
@laf laf commented on an outdated diff Apr 28, 2016
html/includes/forms/schedule-maintenance.inc.php
@@ -11,6 +11,7 @@
* option) any later version. Please see LICENSE.txt at the top level of
* the source code distribution for details.
*/
+header('Content-type: application/json');
if (is_admin() === false) {
die('ERROR: You need to be admin');
@laf laf commented on an outdated diff Apr 28, 2016
html/includes/forms/test-transport.inc.php
@@ -11,6 +11,7 @@
* option) any later version. Please see LICENSE.txt at the top level of
* the source code distribution for details.
*/
+header('Content-type: application/json');
if (is_admin() === false) {
die('ERROR: You need to be admin');
@laf
Member
laf commented Apr 28, 2016

Same on this one.

pblasquez added some commits Apr 28, 2016
@pblasquez pblasquez Merge remote-tracking branch 'refs/remotes/librenms/master' into ajax…
…_headers_4
21e2b45
@pblasquez pblasquez Allow for separate headers upon conditional
4cc9203
@laf laf merged commit 16a5291 into librenms:master Apr 29, 2016

3 checks passed

Auto-Deploy Build finished. No test results found.
Details
Scrutinizer No new issues
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment