Skip to content

Remediate CodeQL alerts #101–#125 via log redaction, SSH trust hardening, and SCIM parser fix - #1

Draft
acespan with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-code-scanning-alerts
Draft

Remediate CodeQL alerts #101–#125 via log redaction, SSH trust hardening, and SCIM parser fix#1
acespan with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-code-scanning-alerts

Conversation

Copilot AI commented Aug 6, 2026

Copy link
Copy Markdown

Addressed the requested code scanning alerts across security skill scripts with minimal, targeted changes. True positives were fixed in-place; alerts assessed as false positives were left unchanged and documented.

  • Sensitive data logging: remove/limit high-risk console output

    • Replaced full report dumps with safe summaries in API/JWT/OpenCTI/DCSync flows.
    • Redacted secret values in Gitleaks output paths.
    • Removed stderr fragments from AD/BloodHound command-failure logs to avoid credential/hash leakage.
  • SSH host key validation: enforce trust-on-known-hosts

    • Replaced permissive Paramiko host-key behavior in rsyslog deployment and vuln-scan scripts.
    • Moved to system host keys + reject-unknown policy.
  • SCIM filter parsing: remove ReDoS-prone regex path

    • Replaced regex-based parse_scim_filter with bounded deterministic parsing (split + structural validation).
  • False positives (no code changes)

    • Alerts: #101, #102, #103, #104, #105, #109, #110, #111, #113, #114, #115.
client = paramiko.SSHClient()
client.load_system_host_keys()
client.set_missing_host_key_policy(paramiko.RejectPolicy())

…hardening

Co-authored-by: acespan <87124399+acespan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix code scanning alerts #101 to #125 Remediate CodeQL alerts #101–#125 via log redaction, SSH trust hardening, and SCIM parser fix Aug 6, 2026
Copilot AI requested a review from acespan August 6, 2026 12:25
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.

2 participants