search

cnpm / cnpmcore

Private NPM Registry for Enterprise
https://npmmirror.com
MIT License
622 stars 84 forks source link

fix: use tar fork version to fix memory leak #625

Closed fengmk2 closed 11 months ago

fengmk2 commented 11 months ago

closes https://github.com/cnpm/cnpmcore/issues/624

fengmk2 commented 11 months ago

先 fork 一个版本临时修复一下。 https://github.com/fengmk2/node-tar/commit/710e2be5905a511a19400a969c202d37d9c9ce0c

codecov[bot] commented 11 months ago

Codecov Report

Attention: 167 lines in your changes are missing coverage. Please review.

Comparison is base (6624a36) 97.45% compared to head (39028f2) 96.83%. Report is 145 commits behind head on master.

Files Patch % Lines
config/config.default.ts 53.03% 31 Missing :warning:
.../port/controller/admin/PaddingVersionController.ts 48.83% 22 Missing :warning:
app/core/service/FixNoPaddingVersionService.ts 54.54% 15 Missing :warning:
app/core/service/PackageSearchService.ts 93.64% 15 Missing :warning:
app/core/service/PackageSyncerService.ts 85.84% 15 Missing :warning:
app/port/schedule/SyncBinaryWorker.ts 21.42% 11 Missing :warning:
app/repository/PackageVersionRepository.ts 87.95% 10 Missing :warning:
app/core/service/PackageVersionFileService.ts 91.57% 8 Missing :warning:
app/core/entity/PaddingSemVer.ts 88.00% 6 Missing :warning:
app/core/service/PackageVersionService.ts 94.91% 6 Missing :warning:
... and 11 more
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #625 +/- ## ========================================== - Coverage 97.45% 96.83% -0.62% ========================================== Files 168 180 +12 Lines 15691 17548 +1857 Branches 2018 2285 +267 ========================================== + Hits 15291 16993 +1702 - Misses 400 555 +155 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.