Skip to content

Audit fix/2026 02 10 remediation#53

Merged
bb-connor merged 15 commits intomainfrom
audit-fix/2026-02-10-remediation
Feb 10, 2026
Merged

Audit fix/2026 02 10 remediation#53
bb-connor merged 15 commits intomainfrom
audit-fix/2026-02-10-remediation

Conversation

@bb-connor
Copy link
Copy Markdown
Collaborator

@bb-connor bb-connor commented Feb 10, 2026

Summary

This PR delivers the Wave 1 + Wave 2 audit remediations for Clawdstrike and adds regression coverage plus remediation evidence docs.

Scope in this PR:

  • CS-AUDIT-001..006 remediation
  • CS-AUDIT2-001..007 remediation
  • Regression tests and hardening around remote extends, CONNECT proxy controls, IRM parsing, async/background limits, and session lock lifecycle
  • Audit closure docs under docs/audits/

What Changed

Remote extends hardening (hush-cli + hushd)

  • Enforced host allowlisting and private/non-public IP blocking on remote resolution paths.
  • Hardened git remote parsing:
    • supports SCP-style remotes (including userless SCP form)
    • rejects unsupported schemes (file:// etc.)
    • avoids SCP misclassification for URL remotes.
  • Added commit/ref validation to reject unsafe tokens (including dash-prefixed refs).
  • Added -- separator placement before user-controlled ref token on git fetch path.

CONNECT proxy + hush run resource bounds

  • Added bounded event queue with explicit drop accounting.
  • Added in-flight proxy connection cap + rejection accounting.
  • Added header-read timeout (slowloris mitigation).
  • Added timeout for forwarded writer HTTP calls.
  • Added CONNECT SNI consistency checks for IP-target CONNECT flow.

IRM and policy robustness (clawdstrike)

  • Filesystem IRM: traversal detection tightened and normalization behavior hardened.
  • Network IRM: switched host extraction to strict URL parsing and normalization.
  • Policy extends: added explicit recursion depth guard.
  • Async guard runtime: bounded background in-flight execution and drop semantics.

Session lock lifecycle (hushd)

  • Added idle lock pruning and churn tests to prevent lock table growth.
  • Tightened lock cleanup semantics around termination paths.

Tests Added / Updated

Includes targeted regressions for:

  • CONNECT/SNI mismatch behavior
  • slowloris and in-flight cap behavior
  • bounded event forwarding behavior
  • IRM traversal and URL spoof parsing
  • remote extends scheme/ref validation and host allowlist enforcement
  • async guard background in-flight limits
  • session lock pruning/churn

Evidence Docs

  • docs/audits/2026-02-rust-security-correctness-audit.md
  • docs/audits/2026-02-10-remediation.md
  • docs/audits/2026-02-10-wave2-remediation.md

Follow-up

Wave 3 findings and pre-release posture/tooling are addressed in:

@bb-connor
Copy link
Copy Markdown
Collaborator Author

@codex

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: b739d96e4c

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@bb-connor bb-connor changed the base branch from feat/sdr-execution to main February 10, 2026 07:32
@bb-connor bb-connor changed the base branch from main to feat/sdr-execution February 10, 2026 07:33
@bb-connor bb-connor force-pushed the audit-fix/2026-02-10-remediation branch from b739d96 to 2128fef Compare February 10, 2026 15:53
@bb-connor bb-connor changed the base branch from feat/sdr-execution to main February 10, 2026 15:53
@bb-connor
Copy link
Copy Markdown
Collaborator Author

@codex

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2128fef9fd

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@bb-connor
Copy link
Copy Markdown
Collaborator Author

@codex

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c27d75752e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@bb-connor
Copy link
Copy Markdown
Collaborator Author

@codex

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: bffe1b10ea

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@bb-connor
Copy link
Copy Markdown
Collaborator Author

@codex

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 854c3a4457

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@bb-connor
Copy link
Copy Markdown
Collaborator Author

@codex

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex Review: Didn't find any major issues. What shall we delve into next?

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@bb-connor
Copy link
Copy Markdown
Collaborator Author

@codex

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 5b10f11fde

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@bb-connor bb-connor merged commit 76ac25c into main Feb 10, 2026
25 checks passed
@bb-connor bb-connor deleted the audit-fix/2026-02-10-remediation branch March 3, 2026 05:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant