mirror of
https://github.com/Cacti/cacti.git
synced 2026-10-23 03:27:02 +09:00
* Port #8166 SNMP credential cache + procedural SNMPv3 to develop Ports the shared SNMPv3 credential cache and procedural SNMPv3 routing from the 1.2.x feature (merged as #8166) onto develop, adapted to develop's refactored SNMP layer. Parts of the 1.2.x change that do not apply to develop are omitted: the cacti_exec() idle-timeout rewrite (develop has no cacti_exec()), the PHP floor bump and composer/doc changes (develop is already 8.3+), and the cmd.php max_oids batching (develop's cmd.php already collects SNMP in-process through persistent sessions, so the per-item spawn the batching removed does not occur). - lib/cache.php: new cross-process \Cacti\Cache\SharedCache (file + POSIX shared-memory + process-local tiers, optional AES-256-GCM sealing). Loaded via lib/snmp.php (the Cacti\Cache\ namespace is not PSR-4 autoloaded). The crypto helpers it seals with already live in develop's lib/auth.php, loaded by global.php, so no duplicate was added. - lib/snmp.php: snmp_get_method() now routes SNMPv3 get/getnext to the procedural snmp3_*() calls when the running PHP supports the configured auth/privacy tokens and no context/engine id is set (SHA-224/384 and AES-192/256[C] stay on the binary before PHP 8.6); the three single-get functions gain the procedural v3 path with \Throwable handling; adds the credential-cache subsystem, protocol-option/MD5-DES helpers, snmp_native_protocol(), snmp_format_agent() and cacti_snmp_get_multi(); the binary v3 auth builder consults the cache when enabled. - poller.php / script_server.php: prime / load the shared cache. - poller_maintenance.php: daily out-of-band cache rebuild on every collector (per-poller lastrun marker). - lib/api_device.php / lib/utility.php: bump snmp_cred_version only on v3 credential add/change (and propagate the token to the affected remote poller). - include/global.php: resolve and create the private cache/ directory. - include/global_settings.php: add 'Enable Credential Cache' and 'Enable MD5 and DES for SNMPv3' settings (both off by default); default new-device v3 auth/privacy to SHA/AES instead of legacy MD5/DES. - host.php / automation_snmp.php: drop MD5/DES from the pickers when disabled and default new records to SHA/AES on upgraded installs. - include/config.php.dist: document $config['cache_dir']. - tests/Unit/Core/Cache/SharedCacheTest.php: SharedCache behaviour coverage. - tests/security/baselines: baseline the two new lib/cache.php sinks. * Incorporate #8171: binary fallback for unsupported SNMPv3 protocols Layers PR #8171 onto the develop port so an SNMPv3 device whose auth/privacy protocols the running PHP's ext-snmp cannot service natively (e.g. AES-256 before PHP 8.6) no longer fatals the poller or device-edit screen. - lib/snmp.php: add CactiSnmpBinarySession, a session stand-in exposing the get/getnext/walk/close interface the cacti_snmp_session_*() helpers use and delegating each call to cacti_snmp_*() -> the Net-SNMP binary. cacti_snmp_session() returns it when php-snmp is absent or the v3 protocol is not natively serviceable, so cmd.php/host.php collect these devices via the binary on pre-8.6 and switch to the in-process path on 8.6 with no caller change. v1/v2 snmpget/snmp2_get now go through snmp_format_agent(). The ERRNO_TIMEOUT check tolerates php-snmp being absent. Drop the vendored classSNMP.php compatibility shim and its include (and the Yandex attribution). - cmd.php: open_snmp_session() returns the down sentinel when a host is already marked down this run, instead of returning an unset (null) [] that cacti_snmp_session_get() then dereferenced (TypeError). - lib/ping.php: ping_snmp() runs the availability check through the procedural cacti_snmp_get()/getnext() path so an unsupported protocol falls back to the binary instead of raising a ValueError in SNMP::setSecurity(). * Incorporate #8172: retire include/vendor/phpsnmp, relocate into lib Follow-up to #8171 on the develop port. After classSNMP.php was removed, the include/vendor/phpsnmp/ directory held only Cacti-owned files; this retires it. - lib/snmp.php: the ext-snmp subclass that carried Cacti's bulk_walk_size / value_output_format (previously vendor/phpsnmp/extension.php) is now declared directly as CactiSnmpNativeSession, guarded by CACTI_PHP_SNMP; the include and the 'use phpsnmp\SNMP' alias are gone, and new SNMP(...) / SNMP::VERSION_* now reference the native \SNMP and the new subclass. - lib/mib_parser.php: moved from include/vendor/phpsnmp/mib_parser.php; lib/mib_cache.php includes it via __DIR__. - Remove include/vendor/phpsnmp/extension.php and index.php (directory retired). - tests: SnmpCoverageTest uses \SNMP::ERRNO_TIMEOUT now that phpsnmp\SNMP is gone; drop the stale include/vendor/phpsnmp/ entry from the vendor-policy allowlist. develop's sink baseline tracks constant-based includes differently than 1.2.x, so no baseline refresh was needed here. * Address PR #8174 review: snmp_get_method params, v3 walk, getnext agent, cache bump, CHANGELOG - lib/snmp.php: restore $php_snmp as the 6th snmp_get_method() parameter and move the new $auth_proto/$priv_proto after it (callers pass them by name), so the positional $php_snmp used by SnmpCoverageTest and others keeps working. - lib/snmp.php: keep every SNMPv3 operation except get/getnext on the Net-SNMP binary - the native walk branch implements only v1/v2, so a v3 walk reaching the procedural gate would have silently returned empty when path_snmpbulkwalk is missing. - lib/snmp.php: cacti_snmp_getnext() v1/v2 branches now use snmp_format_agent() like get/get_raw, so an IPv6 literal / non-default port is bracketed. - lib/api_device.php: bump snmp_cred_version only when the host's v3 membership or one of the cached credential fields actually changed, instead of on every save of a v3 device. - CHANGELOG: move the #8171 entry into the issue block and the #8166 entries into the feature block, matching the established section ordering. * Fix CI: unbracket IPv4/DNS SNMP targets, inline auth-cache SQL columns, apply phpcsfixer standard * Refresh sink inventory baseline after short-array normalization * Restore lib/snmp.php 100% coverage after aligning v3 routing to 1.2.x - update the three SnmpCoverageTest assertions that still encoded develop's pre-port behavior: v3 snmp_get_method now returns PHP, an unsupported- protocol v3 session returns the binary-delegating session object, and the v3 cacti_snmp_get path runs natively - add a v3 noAuthNoPriv user to the snmp-coverage snmpd fixture so native snmp3_get is exercised deterministically - add tests covering the ported subsystems to 100%: shared auth cache, binary-delegating session, protocol pickers, v3 credential args, cacti_snmp_get_multi, and the cacti_get_snmpv3_auth cache path - snmp_auth_cache_load uses a resettable process guard so its enabled and disabled branches are both coverable in a single process * Resolve phpstan errors in the relocated SNMP MIB parser and ported snmp.php - lib/mib_parser.php: type the MibParser properties, add parameter/return types, guard file_get_contents()/preg_split() against false, correct the parse_bracket_token delimiter params (string, not integer), and drop the unreachable return after trigger_error(E_USER_ERROR) - now phpstan clean - lib/mib_cache.php: drop the redundant isset($mp->oids) guard (the typed property is always initialized; $mp->mib is the real check) - lib/snmp.php: type the CactiSnmpBinarySession and CactiSnmpNativeSession properties/methods and pass bool to snmp_set_quick_print() - .phpstan.neon: ignore greaterOrEqual.alwaysFalse alongside the existing greater.alwaysFalse (the PHP_VERSION_ID >= 80600 forward-compat gate) - regenerate phpstan-baseline.neon for the irreducible ext-snmp stub strictness (snmp3_get literal-union protocols, int<-1,max> timeouts) in the ported procedural v3 path, the new lib/cache.php shmop stubs, and the $config array-shape offsets in include/global.php * Make the v3 getter catch coverage net-snmp-build-independent The DES-rejection trick that drove the procedural snmp3_get()/snmp3_getnext() try/catch blocks only works on net-snmp builds compiled without DES; CI's Debian net-snmp accepts DES, so those catches went uncovered (98.3%). - extend the existing native_get injection seam (already on cacti_snmp_get) to cacti_snmp_get_raw, cacti_snmp_getnext, and cacti_snmp_get_multi, and drive the catch blocks with a throwing native getter in the test - mark the seam-less cacti_snmp_session() setSecurity catch with @codeCoverageIgnore (it only throws for a protocol the running net-snmp rejects, which is not portably reproducible in the fixture) * Exclude ext-snmp/PHP-8.6-gated defensive branches from snmp coverage Two branches in lib/snmp.php are unreachable on the PHP 8.3 + ext-snmp coverage runner and were only "covered" locally via opcache dead-code elimination: - snmp_get_method()'s !function_exists('snmp3_get') fallback (ext-snmp always provides snmp3_get) - snmp_php_v3_protocols_supported()'s PHP_VERSION_ID >= 80600 block (SHA-224/384 and AES-192/256 tokens are only accepted on PHP 8.6+) Mark both with @codeCoverageIgnore so the 100% gate is deterministic across runners regardless of opcache settings. * Account for the #8166 SNMPv3 credential-cache gate query in the batching test poller_update_poller_cache_from_buffer() now runs one SELECT per flush to decide whether the flush touched SNMPv3 poller items before bumping the shared credential-cache version (matching 1.2.x). UpdateDataQueryCacheBatchingIntegrationTest counted 4 statements per flush; update it to 5 (and 3 per-row flushes to 15, 1 batched flush to 5). The guard's intent - batching issues fewer statements than per-row flushes - still holds. * Port #8175: group SNMP options under an SNMP Behavior section Mirror the 1.2.x settings reorganization (#8175) on develop. develop already keeps the SNMP options in a dedicated SNMP tab, so add an "SNMP Behavior" spacer there, order the behavior options as md5/des, credential cache, max get size, bulkwalk fetch size, and move "Disable increasing OID Check" out of the Poller tab to sit last in that group. Display-only; no settings added or removed and no behavior change. Adds the "SNMP Behavior" heading to cacti.pot. * fix: complete dependabot htmx 2.0.10 to 2.0.11 bump Dependabot bumped package.json and package-lock.json to htmx.org 2.0.11 (PR 8179) but left the vendored include/js/htmx.js, its version marker, and the pinned SRI constant at 2.0.10. The htmx-filters CI workflow runs npm ci, which regenerates include/js/htmx.js from the 2.0.11 package, so HtmxTest's runtime sha384 no longer matches the HTMX_2_0_10_SRI constant and the integrity assertion fails. Complete the bump: regenerate the vendored include/js/htmx.js to the 2.0.11 build, bump include/js/htmx.js.version to 2.0.11, rename HTMX_2_0_10_SRI to HTMX_2_0_11_SRI with the recomputed sha384 hash, and update the version/integrity assertions in tests/Unit/Ui/Html/HtmxTest.php and tests/HandOff/HtmxLoaderHandOffTest.php. package.json and package-lock.json match the values dependabot already landed on develop. * settings: move SNMP Agent Support under the SNMP Behavior spacer Relocate the enable_snmp_agent setting (SNMP Agent Support Enabled) so it is the first item under the SNMP Behavior spacer instead of sitting in the Poller General group, grouping it with the other SNMP behavior options. * Address PR #8174 review: keep SHA-224/384 on the binary, honor port on transport targets Resolves the two still-valid Copilot review findings on lib/snmp.php that were left unresolved (only marked outdated after later commits shifted their lines). - snmp_php_v3_protocols_supported(): drop SHA224/SHA384 from the native auth allowlist. ext-snmp's SNMP::setSecurity()/snmp3_*() only map MD5/SHA/SHA256/ SHA512 - there is no SHA-224/SHA-384 mapping on any build, including PHP 8.6 (verified against php-src ext/snmp/snmp.c), so routing those tokens to the procedural path raised a ValueError instead of using the binary. They now always fall back to the Net-SNMP binary. The AES-192/256 privacy tokens that ext-snmp does add on 8.6 stay gated behind PHP_VERSION_ID >= 80600. - snmp_format_agent(): a udp/tcp transport-qualified target that omits a port now has the configured non-default port appended (udp6:[2001:db8::1] + 1161 -> udp6:[2001:db8::1]:1161), matching the Net-SNMP binary path. Targets that already carry a port, and unix: sockets (no port), are left untouched. - SnmpCoverageTest: cover the new transport-target branches; lib/snmp.php stays at 100%.
192 lines
8.7 KiB
PHP
192 lines
8.7 KiB
PHP
<?php
|
|
/*
|
|
+-------------------------------------------------------------------------+
|
|
| Copyright (C) 2004-2026 The Cacti Group |
|
|
| |
|
|
| This program is free software; you can redistribute it and/or |
|
|
| modify it under the terms of the GNU General Public License |
|
|
| as published by the Free Software Foundation; either version 2 |
|
|
| of the License, or (at your option) any later version. |
|
|
| |
|
|
| This program is distributed in the hope that it will be useful, |
|
|
| but WITHOUT ANY WARRANTY; without even the implied warranty of |
|
|
| MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the |
|
|
| GNU General Public License for more details. |
|
|
+-------------------------------------------------------------------------+
|
|
| Cacti: The Complete RRDtool-based Graphing Solution |
|
|
+-------------------------------------------------------------------------+
|
|
| This code is designed, written, and maintained by the Cacti Group. See |
|
|
| about.php and/or the AUTHORS file for specific developer information. |
|
|
+-------------------------------------------------------------------------+
|
|
| http://www.cacti.net/ |
|
|
+-------------------------------------------------------------------------+
|
|
*/
|
|
|
|
require_once __DIR__ . '/headers_secure.php';
|
|
|
|
/**
|
|
* Returns true when htmx support is enabled for this install. Reads the
|
|
* htmx_enabled setting row. Default behavior is enabled on develop so the
|
|
* pilot surfaces; flip the setting to 'off' to disable site-wide.
|
|
*
|
|
* Guards on function_exists('read_config_option') so callers during early
|
|
* CLI bootstrap (before the settings table is reachable) get false rather
|
|
* than a fatal call.
|
|
*
|
|
* @return bool
|
|
*/
|
|
function cacti_htmx_enabled(): bool {
|
|
if (!function_exists('read_config_option')) {
|
|
// @codeCoverageIgnoreStart
|
|
// Reached only before include/global.php declares read_config_option
|
|
// (early CLI bootstrap). Every test loads global.php, which declares it
|
|
// process-wide, so this guard cannot be exercised under the test suite.
|
|
return false;
|
|
// @codeCoverageIgnoreEnd
|
|
}
|
|
|
|
$value = read_config_option('htmx_enabled');
|
|
|
|
// When the settings row is absent read_config_option() falls back to the
|
|
// default from include/global_settings.php ('on'), so '' only reaches us
|
|
// when the cached value is explicitly empty (or no default is registered,
|
|
// as in stripped-down test bootstraps). Treat that as disabled so
|
|
// half-initialised installs are safe.
|
|
// The setting is a drop_array with explicit 'on'/'off' values, so treat only
|
|
// an exact 'on' as enabled. Anything else (an empty cached value, a stray
|
|
// '0'/'1', or an explicit 'off') leaves htmx disabled.
|
|
return $value === 'on';
|
|
}
|
|
|
|
/**
|
|
* Returns true when the current request was issued by htmx. The HX-Request
|
|
* header signals that htmx initiated the fetch, so the page can return a
|
|
* fragment instead of a full HTML document.
|
|
*
|
|
* PHP normalises header names to HTTP_ + uppercase + underscores, so
|
|
* HX-Request arrives as HTTP_HX_REQUEST.
|
|
*
|
|
* @return bool
|
|
*/
|
|
function htmx_is_fragment_request(): bool {
|
|
return ($_SERVER['HTTP_HX_REQUEST'] ?? '') === 'true';
|
|
}
|
|
|
|
/**
|
|
* Returns the vendored htmx version string (reads include/js/htmx.js.version).
|
|
* Empty string when the version file is missing.
|
|
*
|
|
* @return string
|
|
*/
|
|
function htmx_version(): string {
|
|
$path = CACTI_PATH_BASE . '/include/js/htmx.js.version';
|
|
|
|
if (!file_exists($path)) {
|
|
return '';
|
|
}
|
|
|
|
return trim((string) file_get_contents($path));
|
|
}
|
|
|
|
/**
|
|
* Subresource Integrity hash pinned to the exact vendored htmx 2.0.11 build
|
|
* (include/js/htmx.js). This is a constant, not a value recomputed from the
|
|
* served file at runtime: a self-referential hash gives no supply-chain
|
|
* protection, since swapping htmx.js would simply re-hash to match it. With
|
|
* a fixed constant the browser rejects any vendored file that no longer matches.
|
|
*
|
|
* On an htmx upgrade, replace the vendored file and recompute this value:
|
|
* php -r 'echo "sha384-".base64_encode(hash_file("sha384","include/js/htmx.js",true));'
|
|
*/
|
|
const HTMX_2_0_11_SRI = 'sha384-gmJEF2eAKY4e+FDN+qtKIivWyb6ANwDB7JUdUybKgQspPKyEX/pIRZG/0uaRoW2C';
|
|
|
|
/**
|
|
* Returns a <script> tag that loads the vendored htmx.js with an SRI
|
|
* integrity attribute and crossorigin="anonymous". Returns empty string when
|
|
* htmx is disabled or the vendored file is absent.
|
|
*
|
|
* hx-boost is intentionally left off the global document. Boost intercepts
|
|
* every anchor and form, which conflicts with Cacti's existing jQuery-driven
|
|
* navigation and AJAX submission model. Pages that want htmx behaviour opt in
|
|
* per-element with explicit hx-* attributes.
|
|
*
|
|
* The integrity attribute uses the HTMX_2_0_11_SRI constant pinned to the
|
|
* vendored build (see that constant for why the hash is not recomputed here).
|
|
*
|
|
* The src is root-absolute (prefixed with CACTI_PATH_URL) so the asset
|
|
* resolves the same way on plugin pages served from subdirectories, matching
|
|
* get_md5_include_js() in lib/functions.php. A document-relative src 404s
|
|
* whenever the current page lives below the Cacti root. The cache-buster
|
|
* query string matches that helper's shape too (path + '?' + hash, no
|
|
* parameter name), and is recomputed on every call rather than cached in a
|
|
* static: get_md5_include_js()/get_theme_paths() do the same, since a static
|
|
* cache would leave long-lived PHP processes (FPM, script_server) serving a
|
|
* stale hash after include/js/htmx.js changes.
|
|
*
|
|
* Two wiring pieces precede the script element:
|
|
* - an htmx-config meta that disables allowEval/allowScriptTags. htmx 2.0.11
|
|
* defaults both to true, which Cacti's CSP (no unsafe-eval) forbids. The
|
|
* meta is read by htmx at load, so it must appear before the script.
|
|
* - an htmx:configRequest listener that adds the csrf-magic token to
|
|
* body-based (POST/PUT/PATCH) htmx requests. Cacti validates the
|
|
* __csrf_magic field on POSTs; layout.js injects it into $.post payloads,
|
|
* but htmx has no such hook, so the first hx-post would otherwise fail
|
|
* CSRF validation. GET and DELETE are excluded because htmx 2.0.11 encodes
|
|
* their parameters into the URL (methodsThatUseUrlParams), which would
|
|
* leak the token into query strings and server logs.
|
|
*
|
|
* @return string
|
|
*/
|
|
function htmx_script_tag(): string {
|
|
if (!cacti_htmx_enabled()) {
|
|
return '';
|
|
}
|
|
|
|
$js_path = CACTI_PATH_BASE . '/include/js/htmx.js';
|
|
|
|
if (!file_exists($js_path)) {
|
|
return '';
|
|
}
|
|
|
|
// md5 is a cache-buster for the asset URL only, recomputed on every call
|
|
// (not cached in a static) so an htmx.js upgrade is visible without a
|
|
// process restart. Integrity is the pinned constant, never derived from
|
|
// the served file (see HTMX_2_0_11_SRI).
|
|
$md5 = md5_file($js_path);
|
|
|
|
// CACTI_PATH_URL is defined by include/global_path.php during a normal
|
|
// bootstrap. Fall back to '/' when the constant is absent (lightweight
|
|
// fixtures that load lib/htmx.php without the full path bootstrap).
|
|
$base = defined('CACTI_PATH_URL') ? CACTI_PATH_URL : '/';
|
|
$url = $base . 'include/js/htmx.js?' . $md5;
|
|
$nonce_attr = CactiSecureHeaders::getNonceAttribute();
|
|
|
|
$config_meta = "<meta name='htmx-config' content='"
|
|
. htmlspecialchars('{"allowEval":false,"allowScriptTags":false}', ENT_QUOTES, 'UTF-8')
|
|
. "'>\n";
|
|
|
|
// htmx:configRequest fires before each request and bubbles to document;
|
|
// listen on document so this works from the <head> where document.body is
|
|
// still null. Mirror layout.js by adding csrfMagicToken under the
|
|
// __csrf_magic field that csrf-magic validates. Only body-based verbs get
|
|
// the token: htmx puts GET and DELETE parameters into the URL, which would
|
|
// leak it (see the function doc).
|
|
$csrf_wiring = "<script type='text/javascript' $nonce_attr>\n"
|
|
. "document.addEventListener('htmx:configRequest', function(evt) {\n"
|
|
. "\tvar verb = String(evt.detail.verb).toLowerCase();\n"
|
|
. "\tif (typeof csrfMagicToken !== 'undefined' && (verb === 'post' || verb === 'put' || verb === 'patch')) {\n"
|
|
. "\t\tevt.detail.parameters['__csrf_magic'] = csrfMagicToken;\n"
|
|
. "\t}\n"
|
|
. "});\n"
|
|
. "</script>\n";
|
|
|
|
$tag = "<script type='text/javascript' $nonce_attr src='" . htmlspecialchars($url, ENT_QUOTES, 'UTF-8') . "'"
|
|
. " integrity='" . HTMX_2_0_11_SRI . "'"
|
|
. " crossorigin='anonymous'></script>\n";
|
|
|
|
// Both wiring pieces precede the script element (see the function doc):
|
|
// the config meta must exist before htmx reads it at load, and registering
|
|
// the listener first means no configRequest can ever fire without it.
|
|
return $config_meta . $csrf_wiring . $tag;
|
|
}
|