Skip to content

fix: bound Censys responses and API outcomes - #28

Merged
phantom-jacob merged 4 commits into
mainfrom
sodle/espm-5228-censys-validation-followup
Aug 4, 2026
Merged

fix: bound Censys responses and API outcomes#28
phantom-jacob merged 4 commits into
mainfrom
sodle/espm-5228-censys-validation-followup

Conversation

@sodle-splunk

@sodle-splunk sodle-splunk commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Summary

  • stream Censys responses through a fixed byte budget before JSON decoding
  • reject search pages that exceed the requested row count and enforce an action-level deadline across pagination
  • validate numeric, textual, malformed, and noncanonical API outcome fields fail closed
  • recognize certificate misses only from an explicit HTTP 404 instead of response text

Validation context

This is a follow-up to merged PR #26 for PAPP-38109 under ESPM-5228. It addresses the remaining source-validation paths for VULN-94540 and VULN-94544.

Quality gate

  • full pre-commit run --all-files passed
  • focused regressions passed: 9 passed
  • compile workflow pending
  • dual-review PSIRT source validation pending on immutable head d4795e7 (full provider SHA to be recorded by reviewers)

Resource bounds and API outcome handling are separate functional commits with matching release-note lines. Do not squash this PR; semantic-release relies on commit history.

Authored by Codex.

@github-actions

github-actions Bot commented Jul 31, 2026

Copy link
Copy Markdown

Merging this PR will release 2.2.6 with the following release notes:

2.2.6 (2026-08-01)

Connector release changes

  • Bound Censys response bodies before JSON decoding and reject oversized search pages.
  • Apply a cumulative deadline with bounded connect and read timeouts across all pages in a Censys search action.
  • Enforce the same wall-clock deadline while Censys response status and headers are received.
  • Validate Censys body status fields fail closed and recognize certificate misses only from HTTP 404.

@sodle-splunk
sodle-splunk force-pushed the sodle/espm-5228-censys-validation-followup branch from d4795e7 to 4fe7462 Compare August 1, 2026 17:04
@sodle-splunk
sodle-splunk marked this pull request as ready for review August 1, 2026 17:44

@phantom-jacob phantom-jacob left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Written by Codex for Jacob: Reviewed and approved at exact head 4fe7462.

@phantom-jacob
phantom-jacob merged commit e312c18 into main Aug 4, 2026
11 of 19 checks passed
@phantom-jacob
phantom-jacob deleted the sodle/espm-5228-censys-validation-followup branch August 4, 2026 00:08
@splunk-soar-semantic-release

Copy link
Copy Markdown

🎉 This PR is included in version 2.2.6 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants