mirror of https://github.com/MISP/MISP
523 lines
23 KiB
PHP
523 lines
23 KiB
PHP
<?php
|
|
App::uses('SyncTool', 'Tools');
|
|
|
|
class SecurityAudit
|
|
{
|
|
const STRONG_PASSWORD_LENGTH = 17;
|
|
|
|
/**
|
|
* @param Server $server
|
|
* @return array
|
|
*/
|
|
public function run(Server $server)
|
|
{
|
|
$output = [];
|
|
|
|
foreach (['config.php', 'database.php', 'email.php'] as $configFile) {
|
|
$perms = fileperms(CONFIG . $configFile);
|
|
if ($perms & 0x0004) {
|
|
$output['File permissions'][] = ['error', __('%s config file is readable for any user.', $configFile)];
|
|
}
|
|
}
|
|
|
|
$redisPassword = Configure::read('MISP.redis_password');
|
|
if (empty($redisPassword)) {
|
|
$output['Redis'][] = ['error', __('Redis password not set.')];
|
|
} else if (strlen($redisPassword) < 32) { // for Redis, password should be stronger
|
|
$output['Redis'][] = [
|
|
'warning',
|
|
__('Redis password is too short, should be at least 32 chars long.'),
|
|
'https://redis.io/topics/security#authentication-feature',
|
|
];
|
|
}
|
|
|
|
$databasePassword = ConnectionManager::getDataSource('default')->config['password'];
|
|
if (empty($databasePassword)) {
|
|
$output['Database'][] = ['error', __('Database password not set.')];
|
|
} else if (strlen($databasePassword) < self::STRONG_PASSWORD_LENGTH) {
|
|
$output['Database'][] = ['warning', __('Database password is too short, should be at least %s chars long.', self::STRONG_PASSWORD_LENGTH)];
|
|
}
|
|
|
|
$passwordPolicyLength = Configure::read('Security.password_policy_length') ?: $server->serverSettings['Security']['password_policy_length']['value'];
|
|
if ($passwordPolicyLength < 8) {
|
|
$output['Password'][] = ['error', __('Minimum password length is set to %s, it is highly advised to increase it.', $passwordPolicyLength)];
|
|
} elseif ($passwordPolicyLength < 12) {
|
|
$output['Password'][] = ['warning', __('Minimum password length is set to %s, consider raising to at least 12 characters.', $passwordPolicyLength)];
|
|
}
|
|
|
|
if (empty(Configure::read('Security.require_password_confirmation'))) {
|
|
$output['Password'][] = [
|
|
'warning',
|
|
__('Password confirmation is not enabled. %s', $server->serverSettings['Security']['require_password_confirmation']['description']),
|
|
];
|
|
}
|
|
if (!empty(Configure::read('Security.auth')) && !Configure::read('Security.auth_enforced')) {
|
|
$output['Login'][] = [
|
|
'hint',
|
|
__('External authentication is enabled, but local accounts will still work. You can disable the ability to log in via local accounts by setting `Security.auth_enforced` to `true`.'),
|
|
];
|
|
}
|
|
|
|
if (empty(Configure::read('Security.disable_browser_cache'))) {
|
|
$output['Browser'][] = [
|
|
'warning',
|
|
__('Browser cache is enabled. An attacker could obtain sensitive data from the user cache. You can disable the cache by setting `Security.disable_browser_cache` to `true`.'),
|
|
];
|
|
}
|
|
if (empty(Configure::read('Security.check_sec_fetch_site_header'))) {
|
|
$output['Browser'][] = [
|
|
'warning',
|
|
__('The MISP server is not checking `Sec-Fetch` HTTP headers. This is a protection mechanism against CSRF used by modern browsers. You can enable this check by setting `Security.check_sec_fetch_site_header` to `true`.'),
|
|
];
|
|
}
|
|
if (empty(Configure::read('Security.csp_enforce'))) {
|
|
$output['Browser'][] = [
|
|
'warning',
|
|
__('Content security policies (CSP) are not enforced. Consider enabling them by setting `Security.csp_enforce` to `true`.'),
|
|
'https://developer.mozilla.org/en-US/docs/Web/HTTP/CSP',
|
|
];
|
|
}
|
|
if (!env('HTTPS') && strpos(Configure::read('MISP.baseurl'), 'https://') === 0) {
|
|
$output['Browser'][] = [
|
|
'error',
|
|
__('MISP base URL is set to https://, but MISP thinks that the connection is insecure. This usually happens when a server is running behind a reverse proxy. By setting `Security.force_https` to `true`, session cookies will be set as Secure and CSP headers will upgrade insecure requests.'),
|
|
];
|
|
}
|
|
$sessionConfig = Configure::read('Session');
|
|
if (isset($sessionConfig['ini']['session.cookie_secure']) && !$sessionConfig['ini']['session.cookie_secure']) {
|
|
$output['Browser'][] = ['error', __('Setting session cookies as not secure is never a good idea.')];
|
|
}
|
|
|
|
if (empty(Configure::read('Security.advanced_authkeys'))) {
|
|
$output['Auth Key'][] = ['warning', __('Consider enabling Advanced Auth Keys, they provide increased security by only storing the API key hashes.')];
|
|
}
|
|
if (Configure::read('Security.allow_unsafe_apikey_named_param')) {
|
|
$output['Auth Key'][] = ['error', __('It is possible to pass API keys via the URL, meaning that the keys can be logged by proxies.')];
|
|
}
|
|
if (empty(Configure::read('Security.do_not_log_authkeys'))) {
|
|
$output['Auth Key'][] = ['warning', __('Auth Key logging is not disabled. Auth Keys in cleartext can be visible in the Audit log.')];
|
|
}
|
|
|
|
$salt = Configure::read('Security.salt');
|
|
if (empty($salt)) {
|
|
$output['Security salt'][] = ['error', __('Salt is not set.')];
|
|
} else if (strlen($salt) < 32) {
|
|
$output['Security salt'][] = ['warning', __('Salt is too short, should contain at least 32 characters.')];
|
|
} else if ($salt === "Rooraenietu8Eeyo<Qu2eeNfterd-dd+") {
|
|
$output['Security salt'][] = ['error', __('Salt is set to the default value.')];
|
|
}
|
|
|
|
if (empty(Configure::read('MISP.log_client_ip'))) {
|
|
$output['Logging'][] = ['warning', __('Logging client IP in audit log is disabled. Logging IP address can help to solve potential security breaches.')];
|
|
}
|
|
if (empty(Configure::read('MISP.log_user_ips'))) {
|
|
$output['Logging'][] = ['warning', __('Logging client IP in Redis is disabled. Logging IP addresses can help investigate potential security breaches.')];
|
|
}
|
|
if (Configure::read('MISP.log_user_ips') && Configure::read('Security.advanced_authkeys') && empty(Configure::read('MISP.log_user_ips_authkeys'))) {
|
|
$output['Logging'][] = [
|
|
'hint',
|
|
__('You can enable the logging of advanced authkeys by setting `MISP.log_user_ips_authkeys` to `true`.'),
|
|
];
|
|
}
|
|
if (empty(Configure::read('Security.username_in_response_header'))) {
|
|
$output['Logging'][] = [
|
|
'hint',
|
|
__('Passing user information to response headers is disabled. This can be useful for logging user info at the reverse proxy level. You can enable it by setting `Security.username_in_response_header` to `true`.'),
|
|
];
|
|
}
|
|
|
|
if (empty(Configure::read('MISP.attachment_scan_module'))) {
|
|
$output['Attachment scanning'][] = ['hint', __('No module for scanning attachments for viruses is currently defined.')];
|
|
}
|
|
|
|
if (Configure::read('debug')) {
|
|
$output['Debug'][] = ['error', __('Debug mode is enabled for all users.')];
|
|
}
|
|
|
|
if (Configure::read('Proxy.host')) {
|
|
$proxyPassword = Configure::read('Proxy.password');
|
|
if (empty($proxyPassword)) {
|
|
$output['Proxy'][] = ['error', __('Proxy password is empty.')];
|
|
} else if (strlen($proxyPassword) < self::STRONG_PASSWORD_LENGTH) {
|
|
$output['Proxy'][] = ['warning', __('Proxy password is too short, should be at least %s chars long.', self::STRONG_PASSWORD_LENGTH)];
|
|
}
|
|
}
|
|
|
|
if (Configure::read('Security.rest_client_enable_arbitrary_urls')) {
|
|
$output['REST client'][] = [
|
|
'hint',
|
|
__('Users can use the REST client to query any remote URL. This is generally not a good idea if your instance is public.')
|
|
];
|
|
}
|
|
|
|
if (Configure::read('Plugins.ZeroMQ_enable')) {
|
|
$zeroMqPassword = Configure::read('Plugins.ZeroMQ_password');
|
|
if (empty($zeroMqPassword)) {
|
|
$output['ZeroMQ'][] = ['error', __('ZeroMQ password is not set.')];
|
|
} else if (strlen($zeroMqPassword) < self::STRONG_PASSWORD_LENGTH) {
|
|
$output['ZeroMQ'][] = ['warning', __('ZeroMQ password is too short, should be at least %s chars long.', self::STRONG_PASSWORD_LENGTH)];
|
|
}
|
|
|
|
$redisPassword = Configure::read('Plugins.ZeroMQ_redis_password');
|
|
if (empty($redisPassword)) {
|
|
$output['ZeroMQ'][] = ['error', __('Redis password is not set.')];
|
|
} else if (strlen($redisPassword) < 32) { // for Redis, password should be stronger
|
|
$output['ZeroMQ'][] = [
|
|
'warning',
|
|
__('Redis password is too short, should be at least 32 chars long.'),
|
|
'https://redis.io/topics/security#authentication-feature',
|
|
];
|
|
}
|
|
}
|
|
|
|
$this->email($output);
|
|
|
|
/*
|
|
* These settings are dangerous and break both the transparency and potential introduce sync issues
|
|
if (!Configure::read('Security.hide_organisation_index_from_users')) {
|
|
$output['MISP'][] = [
|
|
'hint',
|
|
__('Any user can see list of all organisations. You can disable that by setting `Security.hide_organisation_index_from_users` to `true`. %s', $server->serverSettings['Security']['hide_organisation_index_from_users']['description']),
|
|
];
|
|
}
|
|
if (!Configure::read('Security.hide_organisations_in_sharing_groups')) {
|
|
$output['MISP'][] = [
|
|
'hint',
|
|
__('Any user can see list of all organisations in sharing group that user can see. You can disable that by setting `Security.hide_organisations_in_sharing_groups` to `true`. %s', $server->serverSettings['Security']['hide_organisations_in_sharing_groups']['description']),
|
|
];
|
|
}
|
|
*/
|
|
|
|
$this->feeds($output);
|
|
$this->remoteServers($output);
|
|
|
|
try {
|
|
$cakeVersion = $this->getCakeVersion();
|
|
if (version_compare($cakeVersion, '2.10.21', '<')) {
|
|
$output['Dependencies'][] = ['warning', __('CakePHP version %s is outdated.', $cakeVersion)];
|
|
}
|
|
} catch (RuntimeException $e) {}
|
|
|
|
if (version_compare(PHP_VERSION, '7.3.0', '<')) {
|
|
$output['PHP'][] = [
|
|
'warning',
|
|
__('PHP version %s is not supported anymore. It can be still supported by your distribution. ', PHP_VERSION),
|
|
'https://www.php.net/supported-versions.php'
|
|
];
|
|
} else if (version_compare(PHP_VERSION, '7.4.0', '<')) {
|
|
$output['PHP'][] = [
|
|
'hint',
|
|
__('PHP version 7.3 will not be supported after 6 Dec 2021. Even beyond that date, it can be still supported by your distribution.'),
|
|
'https://www.php.net/supported-versions.php'
|
|
];
|
|
}
|
|
if (ini_get('session.use_strict_mode') != 1) {
|
|
$output['PHP'][] = [
|
|
'warning',
|
|
__('Session strict mode is disabled.'),
|
|
'https://www.php.net/manual/en/session.configuration.php#ini.session.use-strict-mode',
|
|
];
|
|
}
|
|
if (empty(ini_get('session.cookie_httponly'))) {
|
|
$output['PHP'][] = ['error', __('Session cookie is not set as HTTP only. Session cookie can be accessed from JavaScript.')];
|
|
}
|
|
if (!in_array(strtolower(ini_get('session.cookie_samesite')), ['strict', 'lax'])) {
|
|
$output['PHP'][] = [
|
|
'error',
|
|
__('Session cookie SameSite parameter is not defined or set to None.'),
|
|
'https://developer.mozilla.org/en-us/docs/Web/HTTP/Headers/Set-Cookie/SameSite',
|
|
];
|
|
}
|
|
$sidLength = ini_get('session.sid_length');
|
|
if ($sidLength !== false && $sidLength < 32) {
|
|
$output['PHP'][] = [
|
|
'warning',
|
|
__('Session ID length is set to %s, at least 32 is recommended.', $sidLength),
|
|
'https://www.php.net/manual/en/session.configuration.php#ini.session.sid-length',
|
|
];
|
|
}
|
|
$sidBits = ini_get('session.sid_bits_per_character');
|
|
if ($sidBits !== false && $sidBits <= 4) {
|
|
$output['PHP'][] = [
|
|
'warning',
|
|
__('Session ID bit per character is set to %s, at least 5 is recommended.', $sidBits),
|
|
'https://www.php.net/manual/en/session.configuration.php#ini.session.sid-bits-per-character',
|
|
];
|
|
}
|
|
|
|
$this->system($server, $output);
|
|
|
|
return $output;
|
|
}
|
|
|
|
private function feeds(array &$output)
|
|
{
|
|
/** @var Feed $feed */
|
|
$feed = ClassRegistry::init('Feed');
|
|
$enabledFeeds = $feed->find('list', [
|
|
'conditions' => [
|
|
'input_source' => 'network',
|
|
'OR' => [
|
|
'enabled' => true,
|
|
'caching_enabled' => true,
|
|
]
|
|
],
|
|
'fields' => ['name', 'url'],
|
|
]);
|
|
foreach ($enabledFeeds as $feedName => $feedUrl) {
|
|
if (substr($feedUrl, 0, strlen('http://')) === 'http://') {
|
|
$output['Feeds'][] = ['warning', __('Feed %s uses insecure (HTTP) connection.', $feedName)];
|
|
}
|
|
}
|
|
}
|
|
|
|
private function remoteServers(array &$output)
|
|
{
|
|
/** @var Server $server */
|
|
$server = ClassRegistry::init('Server');
|
|
$enabledServers = $server->find('all', [
|
|
'conditions' => ['OR' => [
|
|
'push' => true,
|
|
'pull' => true,
|
|
'push_sightings' => true,
|
|
'caching_enabled' => true,
|
|
]],
|
|
'fields' => ['id', 'name', 'url', 'self_signed', 'cert_file', 'client_cert_file'],
|
|
]);
|
|
foreach ($enabledServers as $enabledServer) {
|
|
if (substr($enabledServer['Server']['url'], 0, strlen('http://')) === 'http://') {
|
|
$output['Remote servers'][] = ['warning', __('Server %s uses insecure (HTTP) connection.', $enabledServer['Server']['name'])];
|
|
} else if ($enabledServer['Server']['self_signed']) {
|
|
$output['Remote servers'][] = ['warning', __('Server %s uses self signed certificate. This is considered insecure.', $enabledServer['Server']['name'])];
|
|
}
|
|
|
|
try {
|
|
$parsed = SyncTool::getServerClientCertificateInfo($enabledServer);
|
|
if (isset($parsed['public_key_size_ok']) && !$parsed['public_key_size_ok']) {
|
|
$algo = $parsed['public_key_type'] . " " . $parsed['public_key_size'];
|
|
$output['Remote servers'][] = ['warning', __('Server %s uses weak client certificate (%s).', $enabledServer['Server']['name'], $algo)];
|
|
}
|
|
} catch (Exception $e) {}
|
|
|
|
try {
|
|
$parsed = SyncTool::getServerCaCertificateInfo($enabledServer);
|
|
if (isset($parsed['public_key_size_ok']) && !$parsed['public_key_size_ok']) {
|
|
$algo = $parsed['public_key_type'] . " " . $parsed['public_key_size'];
|
|
$output['Remote servers'][] = ['warning', __('Server %s uses weak CA certificate (%s).', $enabledServer['Server']['name'], $algo)];
|
|
}
|
|
} catch (Exception $e) {}
|
|
}
|
|
}
|
|
|
|
private function email(array &$output)
|
|
{
|
|
$canSignPgp = Configure::read('GnuPG.sign');
|
|
$canSignSmime = Configure::read('SMIME.enabled') &&
|
|
!empty(Configure::read('SMIME.cert_public_sign')) &&
|
|
!empty(Configure::read('SMIME.key_sign'));
|
|
|
|
if (!$canSignPgp && !$canSignSmime) {
|
|
$output['Email'][] = [
|
|
'warning',
|
|
__('Email signing (PGP or S/MIME) is not enabled.')
|
|
];
|
|
}
|
|
|
|
if ($canSignPgp) {
|
|
$gpgKeyPassword = Configure::read('GnuPG.password');
|
|
if (empty($gpgKeyPassword)) {
|
|
$output['Email'][] = ['error', __('PGP private key password is empty.')];
|
|
} else if (strlen($gpgKeyPassword) < self::STRONG_PASSWORD_LENGTH) {
|
|
$output['Email'][] = ['warning', __('PGP private key password is too short, should be at least %s chars long.', self::STRONG_PASSWORD_LENGTH)];
|
|
}
|
|
}
|
|
|
|
if (!Configure::read('GnuPG.bodyonlyencrypted')) {
|
|
$output['Email'][] = [
|
|
'hint',
|
|
__('Full email body with all event information will be sent, even without encryption.')
|
|
];
|
|
}
|
|
|
|
if ($canSignPgp && !Configure::read('GnuPG.obscure_subject')) {
|
|
$output['Email'][] = [
|
|
'hint',
|
|
__('Even for encrypted emails, the email subject will be sent unencrypted. You can change that behaviour by setting `GnuPG.obscure_subject` to `true`.'),
|
|
];
|
|
}
|
|
|
|
App::uses('CakeEmail', 'Network/Email');
|
|
$email = new CakeEmail();
|
|
$emailConfig = $email->config();
|
|
if ($emailConfig['transport'] === 'Smtp' && $emailConfig['port'] == 25 && !$emailConfig['tls']) {
|
|
$output['Email'][] = [
|
|
'warning',
|
|
__('STARTTLS is not enabled.'),
|
|
'https://en.wikipedia.org/wiki/Opportunistic_TLS',
|
|
];
|
|
}
|
|
}
|
|
|
|
private function system(Server $server, array &$output)
|
|
{
|
|
$kernelBuildTime = $this->getKernelBuild();
|
|
if ($kernelBuildTime) {
|
|
$diff = (new DateTime())->diff($kernelBuildTime);
|
|
$diffDays = $diff->format('a');
|
|
if ($diffDays > 300) {
|
|
$output['System'][] = [
|
|
'warning',
|
|
__('Kernel build time was s days ago. This usually means that the system kernel is not updated.', $diffDays),
|
|
];
|
|
}
|
|
}
|
|
|
|
// uptime
|
|
try {
|
|
$since = $this->execute(['uptime', '-s']);
|
|
$since = new DateTime($since);
|
|
$diff = (new DateTime())->diff($since);
|
|
$diffDays = $diff->format('a');
|
|
if ($diffDays > 100) {
|
|
$output['System'][] = [
|
|
'warning',
|
|
__('Uptime of this server is %s days. This usually means that the system kernel is outdated.', $diffDays),
|
|
];
|
|
}
|
|
} catch (Exception $e) {
|
|
}
|
|
|
|
// Python version
|
|
try {
|
|
$pythonVersion = $this->execute([$server->getPythonVersion(), '-V']);
|
|
$parts = explode(' ', $pythonVersion);
|
|
if ($parts[0] !== 'Python') {
|
|
throw new Exception("Invalid python version response: $pythonVersion");
|
|
}
|
|
|
|
if (version_compare($parts[1], '3.6', '<')) {
|
|
$output['System'][] = [
|
|
'warning',
|
|
__('You are using Python %s. This version is not supported anymore, but it can be still supported by your distribution.', $parts[1]),
|
|
'https://endoflife.date/python',
|
|
];
|
|
} else if (version_compare($parts[1], '3.7', '<')) {
|
|
$output['System'][] = [
|
|
'hint',
|
|
__('You are using Python %s. This version will not be supported beyond 23 Dec 2021, but it can be that it is still supported by your distribution.', $parts[1]),
|
|
'https://endoflife.date/python',
|
|
];
|
|
}
|
|
} catch (Exception $e) {
|
|
}
|
|
|
|
$ubuntuVersion = $this->getUbuntuVersion();
|
|
if ($ubuntuVersion) {
|
|
if (in_array($ubuntuVersion, ['14.04', '19.10'])) {
|
|
$output['System'][] = [
|
|
'warning',
|
|
__('You are using Ubuntu %s. This version doesn\'t receive security support anymore.', $ubuntuVersion),
|
|
'https://endoflife.date/ubuntu',
|
|
];
|
|
} else if (in_array($ubuntuVersion, ['16.04'])) {
|
|
$output['System'][] = [
|
|
'hint',
|
|
__('You are using Ubuntu %s. This version will be not supported after 02 Apr 2021.', $ubuntuVersion),
|
|
'https://endoflife.date/ubuntu',
|
|
];
|
|
}
|
|
}
|
|
}
|
|
|
|
private function getKernelBuild()
|
|
{
|
|
if (!php_uname('s') !== 'Linux') {
|
|
return false;
|
|
}
|
|
|
|
$version = php_uname('v');
|
|
if (substr($version, 0, 7) !== '#1 SMP ') {
|
|
return false;
|
|
}
|
|
$buildTime = substr($version, 7);
|
|
return new DateTime($buildTime);
|
|
}
|
|
|
|
private function getUbuntuVersion()
|
|
{
|
|
if (!php_uname('s') !== 'Linux') {
|
|
return false;
|
|
}
|
|
if (!is_readable('/etc/os-release')) {
|
|
return false;
|
|
}
|
|
$content = file_get_contents('/etc/os-release');
|
|
if ($content === false) {
|
|
return false;
|
|
}
|
|
$parsed = parse_ini_string($content);
|
|
if ($parsed === false) {
|
|
return false;
|
|
}
|
|
if (!isset($parsed['NAME'])) {
|
|
return false;
|
|
}
|
|
if ($parsed['NAME'] !== 'Ubuntu') {
|
|
return false;
|
|
}
|
|
if (!isset($parsed['VERSION_ID'])) {
|
|
return false;
|
|
}
|
|
return $parsed['VERSION_ID'];
|
|
}
|
|
|
|
/**
|
|
* @return string
|
|
*/
|
|
private function getCakeVersion()
|
|
{
|
|
$filePath = APP . 'Lib/cakephp/lib/Cake/VERSION.txt';
|
|
$version = file_get_contents($filePath);
|
|
if (!$version) {
|
|
throw new RuntimeException("Could not open CakePHP version file '$filePath'.");
|
|
}
|
|
foreach (explode("\n", $version) as $line) {
|
|
if ($line[0] === '/') {
|
|
continue;
|
|
}
|
|
return trim($line);
|
|
}
|
|
throw new RuntimeException("CakePHP version not found in file '$filePath'.");
|
|
}
|
|
|
|
private function execute(array $command)
|
|
{
|
|
$descriptorspec = [
|
|
1 => ["pipe", "w"], // stdout
|
|
2 => ["pipe", "w"], // stderr
|
|
];
|
|
|
|
$command = implode(' ', $command);
|
|
$process = proc_open($command, $descriptorspec, $pipes);
|
|
if (!$process) {
|
|
throw new Exception("Command '$command' could not be started.");
|
|
}
|
|
|
|
$stdout = stream_get_contents($pipes[1]);
|
|
if ($stdout === false) {
|
|
throw new Exception("Could not get STDOUT of command.");
|
|
}
|
|
fclose($pipes[1]);
|
|
|
|
$stderr = stream_get_contents($pipes[2]);
|
|
fclose($pipes[2]);
|
|
|
|
$returnCode = proc_close($process);
|
|
if ($returnCode !== 0) {
|
|
throw new Exception("Command '$command' return error code $returnCode. STDERR: '$stderr', STDOUT: '$stdout'");
|
|
}
|
|
|
|
return $stdout;
|
|
}
|
|
}
|