Skip to content

Memory Issue #255

@NghiaTranUIT

Description

@NghiaTranUIT

Description

I've noticed more Crash exceptions from Sentry regarding the Out of Memory. We should investigate it

✅ Please read some blogs to understand how to do the Memory Profile in the ElectronJS app.

Acceptance Criteria

Verify the following scenario

  1. Open the Activity monitor to check the Memory Usage at the launch time (No Request in the main table)
  2. Add a * wildcard to the SSL Proxying List
  3. Make a bunch of requests to capture all HTTPS traffic
  4. -> Verify the Memory Usage is increased
  5. Clear Session
  6. Confirm the memory usage is back to normal ✅
  • If the Memory Usage is back to normal level after clearing the Session -> It indicates that there is no leak -> All captured HTTP/HTTPS flows are clean.
  • Otherwise -> It means these capture flows doesn't release properly -> Bug ❌

Verify the IPC listener are removed if the component is unmounted.

  • It's a common mistake that we didn't remove the Listener when the component is unmounted.
    Ref: https://stackoverflow.com/a/76233938/3127477
  • Review all listeners in the both main / renderer processes -> Verify we don't have these mistakes -> If it's -> Fix it ✅

How to review

  • Book a Video Call with me, and explain your process -> How you tackle the problem -> After & Before result -> Screenshot/video in the PR and live demo ✅

Metadata

Metadata

Assignees

Labels

enhancementNew feature or request✅ DoneTicket is resolved

Type

No type

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions