UTDNebula/utd-trends

Surround localStorage gets with a try-catch

Aperta

#634 aperta il 30 mag 2026

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)TypeScript (35 fork)auto 404
Good First IssueType: Internal

Metriche repository

Star
 (28 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Issue

Sentry Issue: TRENDS-5X

In src/modules/usePersistantState.tsx there is no try-catch around the localStorage call:

function setFromStorage() {
      const value = localStorage.getItem(key);
      if (!value) {
        if (useDefaultValue) {
          internalSetState(defaultValueRef.current);
        }
        return;

leading to a:

SecurityError: The operation is insecure.
  at t (src/modules/usePersistantState.tsx:15:21)
  at <anonymous> (src/modules/usePersistantState.tsx:31:5)
...
(45 additional frame(s) were not displayed)

  • check if there are other localStorage calls without a try-catch in our codebase
  • ensure errors are handled

Guida contributor

Direzione di ricerca
Trova tutte le chiamate localStorage nel codebase e avvolgile con blocchi try catch per gestire SecurityError e altre eccezioni.
Tech stack
typescriptreact
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptTypeScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
90