Skip to content

feat: improve SARIF memory usage#386

Open
cx-diogo-rocha wants to merge 3 commits intomasterfrom
AST-140881-dhl-improve-memory-usage
Open

feat: improve SARIF memory usage#386
cx-diogo-rocha wants to merge 3 commits intomasterfrom
AST-140881-dhl-improve-memory-usage

Conversation

@cx-diogo-rocha
Copy link
Copy Markdown
Contributor

Closes #

Proposed Changes

  1. Reduced the amount of time the report was held in memory for Sarif report. Now only in memory once (this can be reduced later with a deeper refactor).
  2. Added a flag to disable console report, since this was another point in the code where the full report was duplicated in memory. We should later stream to console as well even with the flag absent.

Checklist

  • I covered my changes with tests.
  • I Updated the documentation that is affected by my changes:
    • Change in the CLI arguments
    • Change in the configuration file

I submit this contribution under the Apache-2.0 license.

@cx-diogo-rocha cx-diogo-rocha requested a review from a team as a code owner April 23, 2026 14:40
@cx-diogo-rocha cx-diogo-rocha changed the title Ast 140881- Improve SARIF memory usage feat: improve SARIF memory usage Apr 23, 2026
@github-actions
Copy link
Copy Markdown

kics-logo

KICS version: v1.7.13

Category Results
HIGH HIGH 0
MEDIUM MEDIUM 0
LOW LOW 0
INFO INFO 0
TRACE TRACE 0
TOTAL TOTAL 0
Metric Values
Files scanned placeholder 16
Files parsed placeholder 16
Files failed to scan placeholder 0
Total executed queries placeholder 53
Queries failed to execute placeholder 0
Execution time placeholder 1

@github-actions
Copy link
Copy Markdown

Logo
Checkmarx One – Scan Summary & Details8d451735-06f3-4817-a017-a1eab4e91153


New Issues (1) Checkmarx found the following issues in this Pull Request
# Severity Issue Source File / Package Checkmarx Insight
1 HIGH Plain_Text_Transport_Layer_in_Server main.go: 22
detailsThe application starts a server using , in /main.go at line 22. This server will listen and respond to requests made in plain text, without the u...
Attack Vector

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.

1 participant