skip graph.php authentication if configured #3019

Merged
merged 3 commits into from Feb 17, 2016

Projects

None yet

3 participants

@pblasquez
Contributor

Major overhead is saved by skipping graph.php authentication entirely if $config['allow_unauth_graphs'] == 1.

@pblasquez pblasquez Update graph.php
adedd87
@f0o f0o commented on an outdated diff Feb 17, 2016
html/graph.php
@@ -40,8 +40,9 @@
require_once '../includes/rewrites.php';
require_once 'includes/functions.inc.php';
require_once '../includes/rrdtool.inc.php';
-require_once 'includes/authenticate.inc.php';
-
+if($config['allow_unauth_graphs'] == 0) {
@f0o
f0o Feb 17, 2016 Member

Please use != true instead

@f0o f0o added the Blocker label Feb 17, 2016
@pblasquez pblasquez Update graph.php
823177e
@pblasquez pblasquez Update defaults.inc.php
591c40a
@f0o f0o removed the Blocker label Feb 17, 2016
@f0o f0o self-assigned this Feb 17, 2016
@f0o f0o merged commit b5ebaf3 into librenms:master Feb 17, 2016

2 checks passed

Auto-Deploy Build finished. No test results found.
Details
Scrutinizer No new issues
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment