fix: memory leak #99

mfw78 · 2023-10-10T02:42:04Z

Description

This PR addresses potential sources of the memory leak that was observed in staging.

Remove express-prometheus-middlware
Ensure flushing to leveldb on every block
Refactor getLogger to re-use loggers held in the rootLogger within loglevel.

Fixes #98

socket-security · 2023-10-10T02:43:23Z

Removed dependencies detected. Learn more about Socket for GitHub ↗︎

🚮 Removed packages: @types/[email protected], [email protected]

mfw78 added the bug Something isn't working label Oct 10, 2023

mfw78 requested review from anxolin and a team October 10, 2023 02:42

mfw78 self-assigned this Oct 10, 2023

mfw78 marked this pull request as ready for review October 10, 2023 11:45

mfw78 force-pushed the health-probe branch from 056ecab to 2b39b5b Compare October 10, 2023 11:51

Base automatically changed from health-probe to main October 10, 2023 11:53

mfw78 added 3 commits October 10, 2023 11:55

chore: remove express prom middleware

9d4620f

fix: write registry on every block

e9c0659

fix: logging memory leak

d119f01

mfw78 force-pushed the fix-memory-leak branch from 853ecff to d119f01 Compare October 10, 2023 11:55

ahhda approved these changes Oct 11, 2023

View reviewed changes

mfw78 merged commit 5bd3e65 into main Oct 11, 2023
3 checks passed

mfw78 deleted the fix-memory-leak branch October 11, 2023 06:50

github-actions bot locked and limited conversation to collaborators Oct 11, 2023