Skip to content

fix: reject incomplete manifest discovery - #134

Closed
devantler wants to merge 1 commit into
fluxcd:mainfrom
devantler:codex/audit-traversal-544
Closed

devantler wants to merge 1 commit into
fluxcd:mainfrom
devantler:codex/audit-traversal-544

Conversation

@devantler

@devantler devantler commented Oct 6, 2026 •

Copy link
Copy Markdown

Count failed directory traversals as validation errors, including scans that return partial results. Keep validating discovered units while preserving NUL-separated filenames and cleaning up temporary scan storage.

Fixes #133.

Validation: make test build lint and ShellCheck pass. The shell suite covers all five discovery steps, partial results, positive controls and failure handling. A locally built real backend confirms the unreadable-directory failure and readable ConfigMap control.

AI assistance: Codex/gpt-6.1-sol, and Claude Code/claude-opus-5-5 for the rebase. The human author reviewed the contribution and explicitly authorized the DCO sign-off.

@stefanprodan

Copy link
Copy Markdown
Member

@devantler can you please rebase and fix the conflicts? Thanks

Capture directory-scan failures in the parent shell and count incomplete
coverage as a validation error. Keep partial discoveries available for
validation and preserve NUL-delimited filenames. Add shell regressions to
the existing test target.

Assisted-by: Codex/gpt-6.1-sol
Assisted-by: Claude Code/claude-opus-5-5
Signed-off-by: Nikolai Emil Damm <nikolaiemildamm@icloud.com>
@devantler
devantler force-pushed the codex/audit-traversal-544 branch from 9d966c3 to 155c09c Compare October 10, 2026 10:46
@stefanprodan

Copy link
Copy Markdown
Member

This is fixed in main flux-schema discover fails on an unreadable directory.

@devantler

Copy link
Copy Markdown
Author

Nice! Recently, or was I using it wrong?

@devantler

Copy link
Copy Markdown
Author

Ah I see #129

Thank you 🙏

@devantler devantler closed this Oct 10, 2026
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.

Validation action can pass after directory traversal fails

2 participants