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

Map description of OSS Index vulnerabilities #407

Merged
merged 4 commits into from
Mar 17, 2023
Merged

Conversation

nscuro
Copy link
Member

@nscuro nscuro commented Mar 15, 2023

We previously had concerns that including the description would blow the 1MB limit of Kafka events. But now that we use Protobuf AND Snappy compression, even events with ~30 vulnerabilities end up being only ~80kiB. We'll not have any issues whatsoever with event size.

We previously had concerns that including the description would blow the 1MB limit of Kafka events. But now that we use Protobuf AND Snappy compression, even events with ~30 events end up being only ~80kiB. We'll not have any issues whatsoever with event size.

Signed-off-by: nscuro <nscuro@protonmail.com>
@nscuro nscuro added enhancement New feature or request domain/vuln-analysis labels Mar 15, 2023
Signed-off-by: nscuro <nscuro@protonmail.com>
Signed-off-by: nscuro <nscuro@protonmail.com>
Signed-off-by: nscuro <nscuro@protonmail.com>
@sonarcloud
Copy link

sonarcloud bot commented Mar 17, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

80.4% 80.4% Coverage
0.0% 0.0% Duplication

@mehab mehab merged commit 453a24f into main Mar 17, 2023
@nscuro nscuro deleted the map-ossindex-description branch March 17, 2023 16:58
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.

None yet

2 participants