blitzstrike 2.4.46 → 2.4.48

package/CHANGELOG.md

@@ -5,6 +5,39 @@ All notable changes to Blitz Strike are documented here.
 The format follows [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/).
 
+## [2.4.48] — 2026-09-14
+
+### Changed — Precision + consistency (traced to WP core dispatch)
+
+- Grounded the authz/nonce rules in the WordPress core dispatch: `wp_ajax_*` and
+  `admin_post_*` fire ONLY when `is_user_logged_in()`, while `*_nopriv_*` fire
+  unauthenticated. `missing_authz` is now scoped to nopriv handlers that perform
+  a state-changing/privileged operation.
+- Added a DANGEROUS_OP_RE gate: `missing_nonce`/`missing_authz` only fire when
+  the handler actually performs a WRITE (option/meta/user/post/comment writes,
+  file upload/write, `$wpdb->query/insert/update/delete`, `wp_mail`, `wp_redirect`).
+  Read-only handlers (list/search/get) are no longer flagged — removes the CSRF/
+  authz false-positive storm (forminator 90→16, elementor 11→4, wp-fastest-cache
+  15→1).
+- Removed `is_admin()` / `wp_doing_ajax()` from the auth surface — they are
+  request-context checks, not authorization (treating them as auth caused false
+  negatives).
+
+## [2.4.47] — 2026-09-14
+
+### Changed — WordPress integration coverage is now COMPLETE (class-method + direct handlers)
+
+- The WP authz/nonce detector previously only matched STRING handlers
+  (`add_action('wp_ajax_x','fn')`), missing the `array($this,'method')` /
+  `array('Class','method')` form that dominates modern class-based plugins.
+  Now resolves any handler form via `handlerName()` + `functionBody()`.
+- Added raw request-handler detection: `add_action('init'|'admin_init'|
+  'wp_loaded'|'template_redirect', ...)` that reads request input directly —
+  flagged for missing nonce (CSRF) and missing capability (privileged ops).
+- Auth surface widened (`is_admin`, `is_super_admin`, `current_user_can_edit_post`,
+  `wp_doing_ajax`). Benchmark: elementor 0→11, forminator ~3→90, learnpress 1→11
+  authz/nonce findings (class-method handlers now visible).
+
 ## [2.4.46] — 2026-09-14
 
 ### Added — SQL injection (raw string-interpolated query) detector

package/dist/index.js

@@ -56471,43 +56471,58 @@ function functionBody(code, name) {
   }
   return code.slice(open);
 }
+function handlerName(expr) {
+  const quoted = expr.match(/['"]([A-Za-z_][A-Za-z0-9_]*)['"]/g);
+  if (quoted && quoted.length > 0)
+    return quoted[quoted.length - 1].replace(/['"]/g, "");
+  return null;
+}
 var INDIRECT_AUTH = /\b[a-zA-Z0-9_]*(auth|permission|cap|nonce|can_|check|verify|allow)[a-zA-Z0-9_]*\s*\(/;
+var AUTH_RE = /current_user_can|is_user_logged_in|is_super_admin|current_user_can_edit_post|authorize/;
+var NONCE_RE = /wp_verify_nonce|check_ajax_referer|check_admin_referer|verify_nonce/;
+var DANGEROUS_OP_RE = /\b(update_option|add_option|delete_option|update_site_option|delete_site_option|update_post_meta|add_post_meta|delete_post_meta|update_user_meta|add_user_meta|delete_user_meta|update_metadata|add_metadata|delete_metadata|set_transient|delete_transient|wp_insert_post|wp_update_post|wp_delete_post|wp_trash_post|wp_untrash_post|wp_insert_comment|wp_update_comment|wp_delete_comment|wp_insert_user|wp_update_user|wp_delete_user|wp_create_user|wp_set_current_user|set_user_role|wp_set_auth_cookie|wp_mail|wp_new_user_notification|wp_handle_upload|move_uploaded_file|wp_upload_bits|wp_set_object_terms|wp_set_post_terms|wp_schedule_event|wp_clear_scheduled_hook|wp_redirect|file_put_contents|unlink|rename|wpdb->query|wpdb->insert|wpdb->update|wpdb->delete|wpdb->replace)\s*\(/;
 function detectWpAuthz(code, file, out) {
-  const ajaxRe = /add_action\s*\(\s*['"](wp_ajax(?:_nopriv)?_\w+)['"]\s*,\s*['"](\w+)['"]/g;
+  const ajaxRe = /add_action\s*\(\s*['"](wp_ajax(?:_nopriv)?_\w+)['"]\s*,\s*(.+?)\s*\)/g;
   let m;
   while ((m = ajaxRe.exec(code)) !== null) {
     const hook = m[1];
-    const handler = m[2];
+    const handler = handlerName(m[2]);
+    if (!handler)
+      continue;
     const nopriv = hook.startsWith("wp_ajax_nopriv_");
     const body = functionBody(code, handler);
     if (!body)
       continue;
-    const hasCap = /current_user_can|is_user_logged_in|authorize/.test(body) || INDIRECT_AUTH.test(body);
-    const hasNonce = /wp_verify_nonce|check_ajax_referer|check_admin_referer|verify_nonce/.test(body);
+    const hasCap = AUTH_RE.test(body) || INDIRECT_AUTH.test(body);
+    const hasNonce = NONCE_RE.test(body);
+    const dangerous = DANGEROUS_OP_RE.test(body);
     const line = lineNoOf(code, m.index);
-    if (nopriv && !hasCap) {
-      out.push({ file, line, type: "missing_authz", category: "Missing capability check (unauthenticated AJAX handler)", severity: "high", evidence: `add_action('${hook}', '${handler}')`, detail: `Unauthenticated AJAX handler '${handler}()' performs no current_user_can()/authorization check — any unauthenticated visitor can trigger it (auth bypass / privilege escalation).` });
+    if (nopriv && !hasCap && dangerous) {
+      out.push({ file, line, type: "missing_authz", category: "Missing capability check (unauthenticated AJAX handler)", severity: "high", evidence: `add_action('${hook}', ...)`, detail: `Unauthenticated AJAX handler '${handler}()' performs a privileged/state-changing operation with no current_user_can()/authorization check — auth bypass / privilege escalation.` });
     }
-    if (!hasNonce) {
-      out.push({ file, line, type: "missing_nonce", category: "Missing nonce verification", severity: "medium", evidence: `add_action('${hook}', '${handler}')`, detail: `AJAX handler '${handler}()' does not verify a nonce (wp_verify_nonce / check_ajax_referer) — CSRF.` });
+    if (!hasNonce && dangerous) {
+      out.push({ file, line, type: "missing_nonce", category: "Missing nonce verification", severity: "medium", evidence: `add_action('${hook}', ...)`, detail: `AJAX handler '${handler}()' performs a state-changing operation without nonce verification (wp_verify_nonce / check_ajax_referer) — CSRF.` });
     }
   }
-  const adminRe = /add_action\s*\(\s*['"](admin_post(?:_nopriv)?_\w+)['"]\s*,\s*['"](\w+)['"]/g;
+  const adminRe = /add_action\s*\(\s*['"](admin_post(?:_nopriv)?_\w+)['"]\s*,\s*(.+?)\s*\)/g;
   while ((m = adminRe.exec(code)) !== null) {
     const hook = m[1];
-    const handler = m[2];
+    const handler = handlerName(m[2]);
+    if (!handler)
+      continue;
     const nopriv = hook.startsWith("admin_post_nopriv_");
     const body = functionBody(code, handler);
     if (!body)
       continue;
-    const hasCap = /current_user_can|is_user_logged_in|authorize/.test(body) || INDIRECT_AUTH.test(body);
-    const hasNonce = /wp_verify_nonce|check_admin_referer|verify_nonce/.test(body);
+    const hasCap = AUTH_RE.test(body) || INDIRECT_AUTH.test(body);
+    const hasNonce = NONCE_RE.test(body);
+    const dangerous = DANGEROUS_OP_RE.test(body);
     const line = lineNoOf(code, m.index);
-    if (nopriv && !hasCap) {
-      out.push({ file, line, type: "missing_authz", category: "Missing capability check (unauthenticated admin-post handler)", severity: "high", evidence: `add_action('${hook}', '${handler}')`, detail: `Unauthenticated admin-post handler '${handler}()' has no authorization check — auth bypass / privilege escalation.` });
+    if (nopriv && !hasCap && dangerous) {
+      out.push({ file, line, type: "missing_authz", category: "Missing capability check (unauthenticated admin-post handler)", severity: "high", evidence: `add_action('${hook}', ...)`, detail: `Unauthenticated admin-post handler '${handler}()' performs a privileged/state-changing operation with no authorization check — auth bypass / privilege escalation.` });
     }
-    if (!hasNonce) {
-      out.push({ file, line, type: "missing_nonce", category: "Missing nonce verification", severity: "medium", evidence: `add_action('${hook}', '${handler}')`, detail: `admin-post handler '${handler}()' does not verify a nonce — CSRF.` });
+    if (!hasNonce && dangerous) {
+      out.push({ file, line, type: "missing_nonce", category: "Missing nonce verification", severity: "medium", evidence: `add_action('${hook}', ...)`, detail: `admin-post handler '${handler}()' performs a state-changing operation without nonce verification — CSRF.` });
     }
   }
   const restRe = /register_rest_route\s*\(\s*['"]([^'"]+)['"]\s*,\s*['"]([^'"]+)['"]/g;
@@ -56518,6 +56533,28 @@ function detectWpAuthz(code, file, out) {
       continue;
     out.push({ file, line, type: "missing_authz", category: "REST route missing permission_callback", severity: "high", evidence: `register_rest_route('${m[1]}', '${m[2]}')`, detail: `REST route '${m[1]}${m[2]}' has no permission_callback — the endpoint defaults to public (auth bypass).` });
   }
+  const formRe = /add_action\s*\(\s*['"](init|admin_init|wp_loaded|template_redirect|parse_request|wp)['"]\s*,\s*(.+?)\s*\)/g;
+  while ((m = formRe.exec(code)) !== null) {
+    const hook = m[1];
+    const handler = handlerName(m[2]);
+    if (!handler)
+      continue;
+    const body = functionBody(code, handler);
+    if (!body)
+      continue;
+    if (!/\$_(GET|POST|REQUEST|COOKIE|SERVER)\b|php:\/\/input|filter_input|file_get_contents\(['"]php:\/\/input/.test(body))
+      continue;
+    const hasCap = AUTH_RE.test(body) || INDIRECT_AUTH.test(body);
+    const hasNonce = NONCE_RE.test(body);
+    const dangerous = DANGEROUS_OP_RE.test(body);
+    const line = lineNoOf(code, m.index);
+    if (!hasNonce && dangerous) {
+      out.push({ file, line, type: "missing_nonce", category: "Missing nonce verification (direct request handler)", severity: "medium", evidence: `add_action('${hook}', ...)`, detail: `Request handler '${handler}()' (on '${hook}') performs a state-changing operation without nonce verification — CSRF.` });
+    }
+    if (!hasCap && dangerous) {
+      out.push({ file, line, type: "missing_authz", category: "Missing capability check (direct request handler)", severity: "high", evidence: `add_action('${hook}', ...)`, detail: `Request handler '${handler}()' (on '${hook}') performs a privileged/state-changing operation with no capability check — auth bypass / privilege escalation.` });
+    }
+  }
 }
 var ROUTE_DETECTORS = [
   { framework: "laravel", entry: /Route::(?:get|post|put|patch|delete|any|match|resource)\s*\(\s*['"][^'"]+['"]/g, auth: /->middleware\s*\(\s*['"]auth|middleware\s*\(\s*\[[^\]]*['"]auth|->can\s*\(|\$this->authorize|Gate::|authorize\s*\(|@can\b/, window: 2 },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "blitzstrike",
-  "version": "2.4.46",
+  "version": "2.4.48",
   "description": "Blitz Strike — a universal MCP security-audit toolbelt. BLITZ sweeps the attack surface, EAGLE-EYE traces source-to-sink, STRIKE verifies live. 57 attack chains, 130-tool catalog, intelligence data layer. One server, every agent.",
   "type": "module",
   "bin": {