Skip to content

Conversation

@cx-miguel-neiva
Copy link
Contributor

@cx-miguel-neiva cx-miguel-neiva commented Apr 16, 2025

Proposed Changes
Fixed an issue where 2ms was reaching high memory peaks when scanning repositories with a lot of secrets.

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-miguel-neiva cx-miguel-neiva requested a review from a team as a code owner April 16, 2025 10:47
@cx-miguel-neiva cx-miguel-neiva changed the title fix: Decrease memo fix: Improve memory perfomance in large repos Apr 16, 2025
@github-actions
Copy link

github-actions bot commented Apr 16, 2025

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 11
Files parsed placeholder 11
Files failed to scan placeholder 0
Total executed queries placeholder 53
Queries failed to execute placeholder 0
Execution time placeholder 1

@cx-miguel-neiva cx-miguel-neiva changed the title fix: Improve memory perfomance in large repos fix: improve memory perfomance in large repos Apr 16, 2025
@cx-miguel-neiva cx-miguel-neiva merged commit dd1dad8 into master Apr 23, 2025
11 checks passed
@cx-miguel-neiva cx-miguel-neiva deleted the AST-81518-massive-memory-increase branch April 23, 2025 16:45
cx-rogerio-dalot pushed a commit that referenced this pull request Sep 11, 2025
**Proposed Changes**
Fixed an issue where 2ms was reaching high memory peaks when scanning
repositories with a lot of secrets.

**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.

---------

Co-authored-by: Leonardo Fontes <[email protected]>
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.

3 participants