-
Notifications
You must be signed in to change notification settings - Fork 16
path-wildcards: anchored trailing-* + per-endpoint port + R0040 args #807
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
entlein
wants to merge
17
commits into
kubescape:main
Choose a base branch
from
k8sstormcenter:upstream-pr/sbob-path-wildcards
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
17 commits
Select commit
Hold shift + click to select a range
fa7e665
feat(projection): add ExecsByPath composite-key surface to ProjectedC…
entlein 5289c1c
path-wildcards: anchored trailing-* + per-endpoint port + R0040 args
entlein 65c05e2
restoring ap_was_path_opened_with_flags
entlein c71c9a4
apply rabbit feedback: align R0040 args consumer with rc1 final state
entlein 6b5a843
build(go.mod): replace kubescape/storage with sister execs branch
93463de
test: add Test_32_UnexpectedProcessArguments +
entlein f129139
test: add Test_32_UnexpectedProcessArguments + fixtures
entlein 4e4d55a
build(go.mod): drop sister-branch replace; pin kubescape/storage v0.0…
03676d7
fix(tests): de-duplicate Test_32 / file-header / package decl
cbacdd4
build(go.mod): tidy + pin runtime-spec v1.2.1 for -mod=readonly clean
2140392
fix(projection): ExecsByPath becomes path -> []argv-vectors
de16c94
test(mock): populate ExecsByPath in RuleObjectCacheMock projection
c2b7d72
test 27 is for PR 807, test 32 is for PR 805, now swapping them , als…
entlein 86f7a2e
restoring earlier go dependencies
entlein 7683ef0
go.mod dependencies
entlein f260e5f
adding strings in import
entlein e5069fd
rebase attempt 1
entlein File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🛠️ Refactor suggestion | 🟠 Major | ⚡ Quick win
Add TODO comments documenting the temporary nature and removal criteria of these replace directives.
These replace directives downgrade transitive dependencies (stereoscope from v0.1.22→v0.1.9 and runtime-spec from v1.3.0→v1.2.1) to fix build compatibility issues. Without inline documentation explaining why these downgrades are necessary and when they can be removed, this workaround may persist indefinitely after upstream compatibility is restored.
📝 Proposed fix to add removal tracking
📝 Committable suggestion
🤖 Prompt for AI Agents