UTDNebula/utd-trends

Surround localStorage gets with a try-catch

Aberta

#634 aberto em 30 de mai. de 2026

Ver no GitHubGuia do colaborador
 (0 comentário) (0 reação) (0 responsável)TypeScript (35 forks)auto 404
Good First IssueType: Internal

Métricas do repositório

Stars
 (28 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

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

Guia do colaborador

Direção de pesquisa
Encontre todas as chamadas localStorage no codebase e envolva as com blocos try catch para lidar com SecurityError e outras exceções.
Pilha de tecnologia
typescriptreact
Domain
frontend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaScriptTypeScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
90