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

Memory leak when creating multiple instances of api client

Aperta
  • Qualcosa non va?
#657 2 commenti 1 reazione 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
38/100
Tipo di issue
Bug
Chiarezza
Da chiarire
Stato di attività
Tranquilla
Stack tecnologico
nodejs, typescript
Ambito
api, backend

Direzione di ricerca

Non vengono indicati file o test del repository. Inizia tracciando la creazione di Client e il limiter Bottleneck menzionato nel report, quindi riproduci la creazione ripetuta di client confrontando gli snapshot dell’heap. Il lavoro è completato quando le istanze ripetute non causano più una crescita imprevista delle funzioni Bottleneck trattenute e della memoria.

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

Descrizione

Hello,

Recently we spotted memory leak in our Fibery-Hubspot integration and I suspect that sdk is causing it

Image

After looking at heapsnapshots I see there is a huge amount of functions created via Bottleneck

here are urls to the files. Second snapshot was taken few days after the first one

https://qm5o798ggr.ufs.sh/f/hmIrD5vM1qJoIZx1dOE0S938yOiXZVPJxczgKor5b47WmEjt

https://qm5o798ggr.ufs.sh/f/hmIrD5vM1qJooZYpe9LP02MwkEuIciFQtDZzCBOpofxAWjR5

We create new class instance for every client so rate limit calculation is isolated per workspace

Here is the code I use for client initialisation

import {Account} from '@fibery/connector';
import {Client, DEFAULT_LIMITER_OPTIONS} from '@hubspot/api-client';
import {LRUCache} from 'lru-cache';

const clientsPerAccount = new LRUCache<string, Client>({
  max: 300,
});

export const anonHubspotClient = new Client();

export const getHubspotClient = (account: Account) => {
  let client = clientsPerAccount.get(account.name);

  if (client) {
    if (client.config.accessToken !== account.accessToken) {
      client.setAccessToken(account.accessToken);
    }
  } else {
    client = new Client({
      accessToken: account.accessToken,
      // For OAuth apps, each HubSpot account that installs your app is limited to 100 requests every 10 seconds (TEN_SECONDLY_ROLLING)
      // https://developers.hubspot.com/docs/api/usage-details#rate-limits
      limiterOptions: {
        ...DEFAULT_LIMITER_OPTIONS,
        reservoir: 99,
        reservoirRefreshAmount: 99,
        reservoirRefreshInterval: 10000,
      },
      numberOfApiCallRetries: 4,
    });

    clientsPerAccount.set(account.name, client);
  }

  return client;
};

Thanks in advance for looking at it 🙇🏻 Please let me know if you need more details

Lingua principale
TypeScript
Stelle
402
Fork
116
Nessuna PR unita negli ultimi 30g

Preparare l'ambiente

Non abbiamo ancora controllato i file di configurazione di questo progetto. Parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

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 HubSpot/hubspot-api-nodejs

Tutte le issue di HubSpot/hubspot-api-nodejs

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.