Skip to content

Ensure Lurker Does Not Run Out of Memory When Uploading Large Scan Results#2336

Merged
J12934 merged 2 commits intosecureCodeBox:mainfrom
J12934:feature/stream-result-file-without-reading-it-all-at-once
Mar 8, 2024
Merged

Ensure Lurker Does Not Run Out of Memory When Uploading Large Scan Results#2336
J12934 merged 2 commits intosecureCodeBox:mainfrom
J12934:feature/stream-result-file-without-reading-it-all-at-once

Conversation

@J12934
Copy link
Copy Markdown
Member

@J12934 J12934 commented Mar 6, 2024

Description

Stream uploaded file to s3 bucket instead of loading it all upfront into memory

Related to #2327

Checklist

  • Test your changes as thoroughly as possible before you commit them. Preferably, automate your test by unit/integration tests.
  • Make sure that all your commits are signed-off and that you are added to the Contributors file.
  • Make sure that all CI finish successfully.
  • Optional (but appreciated): Make sure that all commits are Verified.

@J12934 J12934 added the enhancement New feature or request label Mar 6, 2024
@J12934 J12934 self-assigned this Mar 6, 2024
@netlify
Copy link
Copy Markdown

netlify bot commented Mar 6, 2024

Deploy Preview for docs-securecodebox canceled.

Name Link
🔨 Latest commit 0d1d938
🔍 Latest deploy log https://app.netlify.com/sites/docs-securecodebox/deploys/65eadc3dc952f800084872ee

…ng it all upfront into memory

Signed-off-by: Jannik Hollenbach <jannik.hollenbach@iteratec.com>
@J12934 J12934 force-pushed the feature/stream-result-file-without-reading-it-all-at-once branch from e731eb6 to 12e0d4c Compare March 8, 2024 09:18
… files from being uploaded correctly

Signed-off-by: Jannik Hollenbach <jannik.hollenbach@iteratec.com>
@J12934 J12934 force-pushed the feature/stream-result-file-without-reading-it-all-at-once branch from 12e0d4c to 0d1d938 Compare March 8, 2024 09:36
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Mar 8, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

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

See analysis details on SonarCloud

@J12934 J12934 merged commit 56f0d8a into secureCodeBox:main Mar 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants