99designs / 99designs/cmdstalk

Resource leak when tube is removed and -all is used

Aperta
#14 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
Go
Stelle
76
Fork
14
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

I was reading the source and thought I would report a tiny bug such that no one runs into it:

[`watchNewTubes()` method](https://github.com/99designs/cmdstalk/blob/fbd704da919d283643e673bd02efcf0221224b92/broker/broker_dispatcher.go#L82-L95) only starts new worker dispatchers, but never closes down preexisting ones if a tube is removed. If you have a lot of tubes being created and destroyed, you will run out of memory eventually.

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Direzione di ricerca

The issue points to the watchNewTubes() method in broker/broker_dispatcher.go. Start by understanding how tube removal is handled elsewhere in the codebase. Look for existing cleanup logic or worker shutdown patterns. Determine what state needs to be tracked to close dispatchers for removed tubes. Running the existing tests will help verify any changes.

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

Valutazione

Stack tecnologico
go
Ambito
backend
Tipo di issue
Bug
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Ferma
Chiarezza
Specificata chiaramente
Idoneità per principianti
50/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.