akiomik / akiomik/nosvelte

useReq() writes to console.error unconditionally

Aperta
#85 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
bug
Lingua principale
TypeScript
Stelle
29
Fork
3
Merge medio
3h 5m
PR unite (30g)
11

Descrizione

`useReq()` logs every stream error to the console unconditionally
(`src/lib/stores/useReq.ts:88-89`):

```ts
error: (e) => {
console.error(e);
```

This is library-side logging with no way for a consumer to opt out, and with
`retry` at its default of 3 it fires once per attempt. A library should not write to
the host application's console without being asked to.

Determined by code reading.

## Direction

Remove it, or route it through an opt-in diagnostics channel that the consumer
subscribes to.

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.