Document and Fix Memory Leak in existing Electron App

General Setup / Briefing

====================

This Electron App generally consists of 3 major parts.

The renderer process starts the main part which is in [login to view URL], then

1) Communication with a NodeJS RestAPI Server happens via [login to view URL]

2) A Tokbox WebRTC Session is established via [login to view URL] when a user connects with Browser to the electron app. This part ([login to view URL]) uses the Tokbox API by using the node module @opentok/client. The electron app streams two cameras.

3) The Hardware is interfaced via hardware-interface.js. The electron app runs on a mini pc that is connected to a driving motor and a servo motor that moves the head of the device where the main camera is attached.

Problem - Memory Leak

===================

There is a test script (that can be provided by me) that runs for about 5h simulating users connecting to the electron app moving motors, moving head, streaming cams for a few seconds more and then disconnect. After another few seconds this process restarts.

By running the above test I have monitored growing memory usage by this electron app. There is a memory leak.

Temporary quick fix for now after each connection I do:

[login to view URL]();

[login to view URL](0);

...which restarts the electron app.

==> I think this cleans the memory, right?

Tasks

======

1) a) Monitor / Debug where the memory leak comes from.

1) b) Also, confirm that:

[login to view URL]();

[login to view URL](0);

...within function relaunchApp() (find this function in [login to view URL]) definitely frees the memory and this quick fix is working.

Please put (j s x) before the title of your proposal starts. I need to see if you read my specs in detail. Thank you. Back to the description:

2) After first investigations there are indicators showing that memory leaks are existing in the Tokbox API. Investigate this issue and fix memory leak, or make memory leak manageable within the application. Do NOT change Tokbox's [login to view URL] ! Because even we have a temporary fix in [login to view URL] this will not be sustainable after next releases.

3) Document occurrences of memory leak in detail and send it to me. I will open a detailed GitHub issue on Tokbox.

4) Find, document and fix other memory leaks (if existing) within the application that is on my part (not within [login to view URL])

JavaScript Node.js

Proje NO: #19796448

Proje hakkında