Ajax headers for devices and pollers #3434

Merged
merged 5 commits into from Apr 29, 2016

Projects

None yet

3 participants

@pblasquez
Contributor

Please see #3113 for context, this is the devices & pollers logical grouping.

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 devices and components
b5b3929
@laf laf commented on an outdated diff Apr 28, 2016
html/includes/forms/parse-device-group.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 Apr 28, 2016 Member

Same again :(

@laf laf commented on an outdated diff Apr 28, 2016
html/includes/forms/parse-poller-groups.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 again :(

pblasquez added some commits Apr 28, 2016
@pblasquez pblasquez Merge remote-tracking branch 'refs/remotes/librenms/master' into ajax…
…_headers_3
de03046
@pblasquez pblasquez Allow for separate headers upon conditional
f3c4dac
@laf laf merged commit 05b0731 into librenms:master Apr 29, 2016

3 checks passed

Auto-Deploy Build finished. No test results found.
Details
Scrutinizer 1 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