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

fix(userspace/sysdig): handle SCAP_FAILURE #2086

Merged
merged 7 commits into from
Apr 19, 2024

Conversation

therealbobo
Copy link
Contributor

Usually, in case of truncated scap files, we don't handle errors. This handle the failure gracefully and just continues in the event loop.

@therealbobo therealbobo force-pushed the fix-trunc-scap branch 3 times, most recently from 13024ee to 3b001a9 Compare April 15, 2024 17:11
@therealbobo therealbobo marked this pull request as draft April 16, 2024 09:41
@therealbobo therealbobo force-pushed the fix-trunc-scap branch 2 times, most recently from a1c5797 to 05f33a3 Compare April 16, 2024 13:40
@therealbobo therealbobo marked this pull request as ready for review April 16, 2024 13:40
Signed-off-by: Roberto Scolaro <roberto.scolaro21@gmail.com>
Signed-off-by: Roberto Scolaro <roberto.scolaro21@gmail.com>
@therealbobo therealbobo force-pushed the fix-trunc-scap branch 6 times, most recently from ac39975 to bcab610 Compare April 18, 2024 08:55
Until now we considered that together with SCAP_EOF rc a valid event
was returned. This is no longer the case.

Signed-off-by: Roberto Scolaro <roberto.scolaro21@gmail.com>
Signed-off-by: Roberto Scolaro <roberto.scolaro21@gmail.com>
Signed-off-by: Roberto Scolaro <roberto.scolaro21@gmail.com>
Signed-off-by: Roberto Scolaro <roberto.scolaro21@gmail.com>
Signed-off-by: Roberto Scolaro <roberto.scolaro21@gmail.com>
Copy link

sonarcloud bot commented Apr 18, 2024

Quality Gate Passed Quality Gate passed

Issues
6 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@therealbobo therealbobo merged commit e7fe148 into draios:dev Apr 19, 2024
11 checks passed
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.

None yet

2 participants