UTDNebula/utd-trends

Surround localStorage gets with a try-catch

Ouverte

#634 ouverte le 30 mai 2026

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)TypeScript (35 forks)auto 404
Good First IssueType: Internal

Métriques du dépôt

Stars
 (28 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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

Guide contributeur

Direction de recherche
Trouvez tous les appels localStorage dans le codebase et enveloppez les avec des blocs try catch pour gérer SecurityError et autres exceptions.
Stack technique
typescriptreact
Domaine
frontend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaScriptTypeScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
90