Skip to content
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

Added rule policy #166

Merged
merged 4 commits into from
Nov 6, 2024
Merged

Added rule policy #166

merged 4 commits into from
Nov 6, 2024

Conversation

afek854
Copy link
Contributor

@afek854 afek854 commented Nov 4, 2024

Sorry, we do not accept changes directly against this repository. Please see
CONTRIBUTING.md for information on where and how to contribute instead.

Copy link

github-actions bot commented Nov 4, 2024

Summary:

  • License scan: success
  • Credentials scan: failure
  • Vulnerabilities scan: success
  • Unit test: success
  • Go linting: success

@afek854 afek854 changed the title Added rule restriction Added rule policy Nov 4, 2024
@afek854 afek854 force-pushed the feature/restrict_rule_by_process branch 2 times, most recently from d3e2a65 to 90bb205 Compare November 4, 2024 09:42
Copy link

github-actions bot commented Nov 4, 2024

Summary:

  • License scan: success
  • Credentials scan: failure
  • Vulnerabilities scan: success
  • Unit test: success
  • Go linting: success

@afek854 afek854 force-pushed the feature/restrict_rule_by_process branch from 90bb205 to 2142149 Compare November 4, 2024 09:57
matthyx
matthyx previously approved these changes Nov 4, 2024
Copy link

github-actions bot commented Nov 4, 2024

Summary:

  • License scan: success
  • Credentials scan: failure
  • Vulnerabilities scan: success
  • Unit test: success
  • Go linting: success

Copy link

github-actions bot commented Nov 4, 2024

Summary:

  • License scan: success
  • Credentials scan: failure
  • Vulnerabilities scan: success
  • Unit test: success
  • Go linting: success

Copy link

github-actions bot commented Nov 5, 2024

Summary:

  • License scan: success
  • Credentials scan: failure
  • Vulnerabilities scan: success
  • Unit test: success
  • Go linting: success

@@ -568,6 +571,13 @@ type Syscall struct {
Args []*Arg `json:"args,omitempty" protobuf:"bytes,4,rep,name=args"`
}

type RulePolicy struct {
// +patchStrategy=merge
Copy link
Contributor

Choose a reason for hiding this comment

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

this is useless for JSON patches

@matthyx matthyx added the release label Nov 6, 2024
Copy link

github-actions bot commented Nov 6, 2024

Summary:

  • License scan: success
  • Credentials scan: failure
  • Vulnerabilities scan: success
  • Unit test: success
  • Go linting: success

@afek854 afek854 merged commit ebcd04e into main Nov 6, 2024
6 checks passed
@matthyx matthyx deleted the feature/restrict_rule_by_process branch December 5, 2024 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants