Skip to content

Commit

Permalink
Merge pull request #34 from silverstripe-labs/public-access
Browse files Browse the repository at this point in the history
Allow public access to the dev/health url and change response to 500
  • Loading branch information
Damian Mooyman authored Jul 12, 2016
2 parents a19fcb4 + f48efcf commit a8c0963
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
2 changes: 1 addition & 1 deletion _config/routes.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,6 @@ After: 'framework/*','cms/*'
---
Director:
rules:
'dev/health': 'DevHealthController'
'health/check': 'DevHealthController'
'dev/check/$Suite': 'DevCheckController'

3 changes: 1 addition & 2 deletions code/DevHealthController.php
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,7 @@ public function index()
// health check does not require permission to run

$checker = new EnvironmentChecker('health', 'Site health');
$checker->init('');
$checker->setErrorCode(404);
$checker->setErrorCode(500);

return $checker;
}
Expand Down

0 comments on commit a8c0963

Please sign in to comment.