Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

Memory Leak

Aperta
#32 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
35/100
Tipo di issue
Bug
Chiarezza
Abbastanza chiara
Stato di attività
Ferma
Stack tecnologico
bun, deno, javascript
Ambito
cryptography

Direzione di ricerca

Riproduci la crescita dell'heap segnalata chiamando createHmac in Bun e Deno, confrontandola con la soluzione alternativa che consiste nell'eliminare la proprietà. Esamina prima Legacy.js e il comportamento delle dipendenze del pacchetto; il lavoro è completato quando chiamate HMAC ripetute non mostrano più l'aumento di memoria segnalato senza richiedere l'eliminazione manuale della proprietà.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Hi everyone, I've used this package in Bun (where it is used as part of the crypto module) and in Deno (importing it manually with npm:create-hmac).

Through using it and measuring performance, I noticed that this library creates memory leak when used as so:

const tokenString = `${secret}:${timestamp}`;

 const hmac = createHmac("md5", secret);
 const token = hmac.update(tokenString).digest("hex");
  
  return {
    token,
    timestamp: timestamp,
  };

despite the hmac getting freed from the scope and not stored in any global object / event handler, I could notice that the memory of the project would increased significantly. My test was calling this function once per second, and log the heapsize every 30s. Nothing else was happening in the code. Yet the memory increased every time (5MB for every 30s measurement).

However, the following code does not generate the memory leak:

 const tokenString = `${secret}:${timestamp}`;

  const hmac = createHmac("md5", secret);
  const token = hmac.update(tokenString).digest("hex");

  for (const property in hmac) {
    delete hmac[property as keyof typeof hmac];
  }

  return {
    token,
    timestamp: timestamp,
  };

I'm not sure if the issues lies in your own code (could not see anything incriminating in Legacy.js), or in a dependency you are using, but it is certainly there. I'm also not sure there is anything you can do about it as you need to support the nodeJS crypto signature so it's not like you can add a destroy function.

But I figured it was worth raising as an issue with this working solution in case anyone else falls down the same rabbit hole.

Lingua principale
JavaScript
Stelle
60
Fork
22
Nessuna PR unita negli ultimi 30g

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di browserify/createHmac

Tutte le issue di browserify/createHmac

Issue simili

Altre issue su JavaScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.