Skip to content

fix: remediate Flashpoint connector findings - #61

Merged
sodle-splunk merged 7 commits into
mainfrom
sodle/fix-flashpoint-vulnerabilities
Jul 21, 2026
Merged

sodle-splunk merged 7 commits into
mainfrom
sodle/fix-flashpoint-vulnerabilities

Conversation

@sodle-splunk

@sodle-splunk sodle-splunk commented Jul 18, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • bind stored credentials to the asset-configured Git endpoint
  • constrain repository names and clone destinations to connector state
  • reject file paths that escape a repository for every file operation
  • reject commit identities that can inject Git config sections
  • pin SSH server host keys and fail closed when no key is configured

Breaking change

Existing SSH assets must configure ssh_host_key with the trusted server key in known_hosts format.

Tracking

Validation

  • full pre-commit suite passed outside the sandbox
  • Docker-backed dependency packaging passed
  • static tests, app linter, Ruff, Semgrep, and secret scanning passed
  • hosted pre-commit and compile are required before marking ready

This pull request was authored by Codex with AI assistance.

Tracking

  • PAPP: PAPP-38187
  • PSAAS: PSAAS-30533, PSAAS-30583, PSAAS-30617, PSAAS-30625, PSAAS-30811
  • Written by Codex.

Refs PSAAS-30533.

Written by Codex.
Refs PSAAS-30583.

Written by Codex.
Refs PSAAS-30617.

Written by Codex.
Refs PSAAS-30625.

Written by Codex.
Refs PSAAS-30811.

SSH repository connections now require a configured trusted server host key.

BREAKING CHANGE: SSH repository connections without a configured ssh_host_key now fail closed.

Written by Codex.
@github-actions

github-actions Bot commented Jul 18, 2026 •

Copy link
Copy Markdown

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

5.0.0 (2026-07-18)

Connector release changes

  • Prevent asset credentials from being sent to a caller-selected repository endpoint.
  • Constrain repository names and clone destinations to the connector state directory.
  • Reject out-of-repository file paths before add, update, or delete operations.
  • Reject Git commit identities that could inject configuration sections.
  • Require a pinned server host key for SSH repository connections.

@sodle-splunk
sodle-splunk marked this pull request as ready for review July 18, 2026 06:19
Keep one generated list operator per security fix so semantic-release renders clean changelog entries.\n\nWritten by Codex.

@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.

Approved by Codex after reviewing draft status, release-note formatting, code changes, and associated ticket scope.

@sodle-splunk
sodle-splunk merged commit 2f97007 into main Jul 21, 2026
10 of 19 checks passed
@sodle-splunk
sodle-splunk deleted the sodle/fix-flashpoint-vulnerabilities branch July 21, 2026 15:57
@splunk-soar-semantic-release

Copy link
Copy Markdown

🎉 This PR is included in version 5.0.0 🎉

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