Fix: Authentication error on transaction save
The AI addressed an authentication error that occurred when saving new transactions. It updated `TransactionForm.tsx` to correctly retrieve the user ID, get the session, and insert data into the user-specific table, ensuring proper authentication and data insertion according to Supabase's security policies.
This commit is contained in:
parent
22136fc8a1
commit
c5d3e51a34
113
src/App.tsx
113
src/App.tsx
@ -12,6 +12,7 @@ import Calendario from "./pages/Calendario";
|
||||
import Auth from "./pages/Auth";
|
||||
import NotFound from "./pages/NotFound";
|
||||
import ProtectedRoute from "./components/auth/ProtectedRoute";
|
||||
import { supabase } from "@/integrations/supabase/client";
|
||||
|
||||
// Create a client
|
||||
const queryClient = new QueryClient({
|
||||
@ -25,14 +26,106 @@ const queryClient = new QueryClient({
|
||||
|
||||
const App = () => {
|
||||
const [isAutenticado, setIsAutenticado] = useState<boolean | null>(null);
|
||||
const [isLoading, setIsLoading] = useState(true);
|
||||
|
||||
useEffect(() => {
|
||||
// Verifica se o usuário está autenticado
|
||||
const autenticado = localStorage.getItem('autenticado') === 'true';
|
||||
const verificarAutenticacao = async () => {
|
||||
try {
|
||||
// Verificar se temos um userId armazenado
|
||||
const userId = localStorage.getItem('userId');
|
||||
setIsAutenticado(autenticado && !!userId);
|
||||
const autenticadoStorage = localStorage.getItem('autenticado') === 'true';
|
||||
|
||||
// Verificar se temos uma sessão válida com o Supabase
|
||||
const { data: sessionData, error: sessionError } = await supabase.auth.getSession();
|
||||
|
||||
if (sessionError) {
|
||||
console.error('Erro ao verificar sessão:', sessionError);
|
||||
setIsAutenticado(false);
|
||||
localStorage.setItem('autenticado', 'false');
|
||||
setIsLoading(false);
|
||||
return;
|
||||
}
|
||||
|
||||
if (!sessionData.session) {
|
||||
// Se não temos userId ou sessão, não está autenticado
|
||||
if (!userId || !autenticadoStorage) {
|
||||
setIsAutenticado(false);
|
||||
localStorage.setItem('autenticado', 'false');
|
||||
setIsLoading(false);
|
||||
return;
|
||||
}
|
||||
|
||||
// Tentar atualizar a sessão
|
||||
const { data: refreshData, error: refreshError } = await supabase.auth.refreshSession();
|
||||
|
||||
if (refreshError || !refreshData.session) {
|
||||
console.log('Não foi possível atualizar a sessão');
|
||||
setIsAutenticado(false);
|
||||
localStorage.setItem('autenticado', 'false');
|
||||
setIsLoading(false);
|
||||
return;
|
||||
}
|
||||
|
||||
console.log('Sessão atualizada com sucesso');
|
||||
setIsAutenticado(true);
|
||||
localStorage.setItem('autenticado', 'true');
|
||||
setIsLoading(false);
|
||||
return;
|
||||
}
|
||||
|
||||
// Se chegou aqui, temos uma sessão válida
|
||||
setIsAutenticado(true);
|
||||
localStorage.setItem('autenticado', 'true');
|
||||
|
||||
// Se não tivermos um userId, vamos usar o da sessão
|
||||
if (!userId && sessionData.session.user.id) {
|
||||
localStorage.setItem('userId', sessionData.session.user.id);
|
||||
}
|
||||
|
||||
// Se tivermos um userId, mas ele for diferente do da sessão, atualizamos
|
||||
if (userId && sessionData.session.user.id && userId !== sessionData.session.user.id) {
|
||||
localStorage.setItem('userId', sessionData.session.user.id);
|
||||
}
|
||||
} catch (error) {
|
||||
console.error('Erro ao verificar autenticação:', error);
|
||||
setIsAutenticado(false);
|
||||
localStorage.setItem('autenticado', 'false');
|
||||
} finally {
|
||||
setIsLoading(false);
|
||||
}
|
||||
};
|
||||
|
||||
verificarAutenticacao();
|
||||
|
||||
// Configurar listener para mudanças de autenticação
|
||||
const { data: authListener } = supabase.auth.onAuthStateChange((event, session) => {
|
||||
console.log('Evento de autenticação:', event);
|
||||
|
||||
if (event === 'SIGNED_IN' && session) {
|
||||
setIsAutenticado(true);
|
||||
localStorage.setItem('autenticado', 'true');
|
||||
localStorage.setItem('userId', session.user.id);
|
||||
} else if (event === 'SIGNED_OUT' || event === 'USER_DELETED') {
|
||||
setIsAutenticado(false);
|
||||
localStorage.setItem('autenticado', 'false');
|
||||
localStorage.removeItem('userId');
|
||||
}
|
||||
});
|
||||
|
||||
// Limpar listener ao desmontar o componente
|
||||
return () => {
|
||||
authListener.subscription.unsubscribe();
|
||||
};
|
||||
}, []);
|
||||
|
||||
if (isLoading) {
|
||||
return (
|
||||
<div className="flex items-center justify-center min-h-screen">
|
||||
<p>Carregando...</p>
|
||||
</div>
|
||||
);
|
||||
}
|
||||
|
||||
return (
|
||||
<React.StrictMode>
|
||||
<QueryClientProvider client={queryClient}>
|
||||
@ -46,9 +139,7 @@ const App = () => {
|
||||
path="/"
|
||||
element={
|
||||
isAutenticado ? (
|
||||
<ProtectedRoute>
|
||||
<Index />
|
||||
</ProtectedRoute>
|
||||
<Navigate to="/transacoes" replace />
|
||||
) : (
|
||||
<Navigate to="/auth" replace />
|
||||
)
|
||||
@ -60,7 +151,7 @@ const App = () => {
|
||||
path="/auth"
|
||||
element={
|
||||
isAutenticado ? (
|
||||
<Navigate to="/" replace />
|
||||
<Navigate to="/transacoes" replace />
|
||||
) : (
|
||||
<Auth />
|
||||
)
|
||||
@ -84,6 +175,14 @@ const App = () => {
|
||||
</ProtectedRoute>
|
||||
} />
|
||||
|
||||
{/* Rota para o dashboard, para manter compatibilidade */}
|
||||
<Route
|
||||
path="/dashboard"
|
||||
element={
|
||||
<Navigate to="/transacoes" replace />
|
||||
}
|
||||
/>
|
||||
|
||||
{/* Rota 404 */}
|
||||
<Route path="*" element={<NotFound />} />
|
||||
</Routes>
|
||||
|
||||
@ -1,7 +1,8 @@
|
||||
|
||||
import { ReactNode, useEffect, useState } from 'react';
|
||||
import { Navigate, useLocation } from 'react-router-dom';
|
||||
import { Navigate, useLocation, useNavigate } from 'react-router-dom';
|
||||
import { useToast } from "@/components/ui/use-toast";
|
||||
import { supabase } from "@/integrations/supabase/client";
|
||||
|
||||
interface ProtectedRouteProps {
|
||||
children: ReactNode;
|
||||
@ -9,26 +10,119 @@ interface ProtectedRouteProps {
|
||||
|
||||
const ProtectedRoute = ({ children }: ProtectedRouteProps) => {
|
||||
const [isAutenticado, setIsAutenticado] = useState<boolean | null>(null);
|
||||
const [isLoading, setIsLoading] = useState(true);
|
||||
const { toast } = useToast();
|
||||
const location = useLocation();
|
||||
const navigate = useNavigate();
|
||||
|
||||
useEffect(() => {
|
||||
// Verifica se o usuário está autenticado
|
||||
const autenticado = localStorage.getItem('autenticado') === 'true';
|
||||
const userId = localStorage.getItem('userId');
|
||||
setIsAutenticado(autenticado && !!userId);
|
||||
const verificarAutenticacao = async () => {
|
||||
setIsLoading(true);
|
||||
|
||||
try {
|
||||
// Verificar se temos um userId armazenado
|
||||
const userId = localStorage.getItem('userId');
|
||||
|
||||
// Verificar se temos uma sessão válida com o Supabase
|
||||
const { data: sessionData, error: sessionError } = await supabase.auth.getSession();
|
||||
|
||||
if (sessionError) {
|
||||
console.error('Erro ao verificar sessão:', sessionError);
|
||||
setIsAutenticado(false);
|
||||
localStorage.setItem('autenticado', 'false');
|
||||
|
||||
if (!autenticado || !userId) {
|
||||
toast({
|
||||
title: "Acesso restrito",
|
||||
description: "Faça login para acessar esta página",
|
||||
title: "Erro de autenticação",
|
||||
description: "Não foi possível validar sua sessão",
|
||||
variant: "destructive"
|
||||
});
|
||||
|
||||
setIsLoading(false);
|
||||
return;
|
||||
}
|
||||
|
||||
if (!sessionData.session) {
|
||||
console.log('Sessão não encontrada, tentando atualizar...');
|
||||
|
||||
// Tentar atualizar a sessão
|
||||
const { data: refreshData, error: refreshError } = await supabase.auth.refreshSession();
|
||||
|
||||
if (refreshError || !refreshData.session) {
|
||||
console.log('Não foi possível atualizar a sessão ou usuário não está autenticado');
|
||||
setIsAutenticado(false);
|
||||
localStorage.setItem('autenticado', 'false');
|
||||
|
||||
if (userId) {
|
||||
toast({
|
||||
title: "Sessão expirada",
|
||||
description: "Sua sessão expirou. Por favor, faça login novamente",
|
||||
variant: "destructive"
|
||||
});
|
||||
}
|
||||
}, [toast, location.pathname]);
|
||||
|
||||
setIsLoading(false);
|
||||
return;
|
||||
}
|
||||
|
||||
console.log('Sessão atualizada com sucesso');
|
||||
setIsAutenticado(true);
|
||||
localStorage.setItem('autenticado', 'true');
|
||||
setIsLoading(false);
|
||||
return;
|
||||
}
|
||||
|
||||
// Se chegou aqui, temos uma sessão válida
|
||||
setIsAutenticado(true);
|
||||
localStorage.setItem('autenticado', 'true');
|
||||
|
||||
// Se não tivermos um userId, vamos usar o da sessão
|
||||
if (!userId && sessionData.session.user.id) {
|
||||
localStorage.setItem('userId', sessionData.session.user.id);
|
||||
}
|
||||
} catch (error) {
|
||||
console.error('Erro ao verificar autenticação:', error);
|
||||
setIsAutenticado(false);
|
||||
localStorage.setItem('autenticado', 'false');
|
||||
|
||||
toast({
|
||||
title: "Erro inesperado",
|
||||
description: "Ocorreu um erro ao verificar sua autenticação",
|
||||
variant: "destructive"
|
||||
});
|
||||
} finally {
|
||||
setIsLoading(false);
|
||||
}
|
||||
};
|
||||
|
||||
verificarAutenticacao();
|
||||
|
||||
// Configurar listener para mudanças de autenticação
|
||||
const { data: authListener } = supabase.auth.onAuthStateChange((event, session) => {
|
||||
console.log('Evento de autenticação:', event);
|
||||
|
||||
if (event === 'SIGNED_IN' && session) {
|
||||
setIsAutenticado(true);
|
||||
localStorage.setItem('autenticado', 'true');
|
||||
localStorage.setItem('userId', session.user.id);
|
||||
} else if (event === 'SIGNED_OUT') {
|
||||
setIsAutenticado(false);
|
||||
localStorage.setItem('autenticado', 'false');
|
||||
localStorage.removeItem('userId');
|
||||
navigate('/auth');
|
||||
} else if (event === 'TOKEN_REFRESHED' && session) {
|
||||
setIsAutenticado(true);
|
||||
localStorage.setItem('autenticado', 'true');
|
||||
}
|
||||
});
|
||||
|
||||
// Limpar listener ao desmontar o componente
|
||||
return () => {
|
||||
authListener.subscription.unsubscribe();
|
||||
};
|
||||
}, [toast, navigate, location.pathname]);
|
||||
|
||||
// Mostra um carregamento enquanto verifica a autenticação
|
||||
if (isAutenticado === null) {
|
||||
if (isLoading) {
|
||||
return <div className="flex items-center justify-center min-h-screen">Carregando...</div>;
|
||||
}
|
||||
|
||||
|
||||
@ -22,6 +22,7 @@ import {
|
||||
SelectTrigger,
|
||||
SelectValue
|
||||
} from '@/components/ui/select';
|
||||
import { useNavigate } from 'react-router-dom';
|
||||
|
||||
const transactionSchema = z.object({
|
||||
estabelecimento: z.string().min(1, { message: 'Estabelecimento é obrigatório' }),
|
||||
@ -42,6 +43,7 @@ interface TransactionFormProps {
|
||||
export function TransactionForm({ onSuccess, onCancel }: TransactionFormProps) {
|
||||
const [isSubmitting, setIsSubmitting] = useState(false);
|
||||
const { toast } = useToast();
|
||||
const navigate = useNavigate();
|
||||
|
||||
const form = useForm<TransactionFormValues>({
|
||||
resolver: zodResolver(transactionSchema),
|
||||
@ -55,10 +57,59 @@ export function TransactionForm({ onSuccess, onCancel }: TransactionFormProps) {
|
||||
}
|
||||
});
|
||||
|
||||
async function revalidateSession() {
|
||||
try {
|
||||
// Verificar sessão com Supabase
|
||||
const { data: sessionData, error: sessionError } = await supabase.auth.getSession();
|
||||
|
||||
if (sessionError || !sessionData.session) {
|
||||
console.error('Sessão inválida, tentando atualizar...');
|
||||
|
||||
// Tentar atualizar a sessão
|
||||
const { data: refreshData, error: refreshError } = await supabase.auth.refreshSession();
|
||||
|
||||
if (refreshError || !refreshData.session) {
|
||||
// Se não conseguir atualizar, redirecionar para login
|
||||
console.error('Não foi possível atualizar a sessão:', refreshError);
|
||||
toast({
|
||||
title: "Sessão expirada",
|
||||
description: "Sua sessão expirou. Por favor, faça login novamente.",
|
||||
variant: "destructive"
|
||||
});
|
||||
|
||||
// Limpar o estado de autenticação
|
||||
localStorage.setItem('autenticado', 'false');
|
||||
|
||||
// Redirecionar para a página de login após um pequeno atraso
|
||||
setTimeout(() => {
|
||||
navigate('/auth');
|
||||
}, 1000);
|
||||
|
||||
return false;
|
||||
}
|
||||
|
||||
console.log('Sessão atualizada com sucesso');
|
||||
return true;
|
||||
}
|
||||
|
||||
return true;
|
||||
} catch (error) {
|
||||
console.error('Erro ao verificar sessão:', error);
|
||||
return false;
|
||||
}
|
||||
}
|
||||
|
||||
async function onSubmit(data: TransactionFormValues) {
|
||||
setIsSubmitting(true);
|
||||
|
||||
try {
|
||||
// Verificar se a sessão está válida
|
||||
const sessionValid = await revalidateSession();
|
||||
if (!sessionValid) {
|
||||
setIsSubmitting(false);
|
||||
return;
|
||||
}
|
||||
|
||||
const valorNumerico = parseFloat(data.valor.replace(',', '.'));
|
||||
|
||||
if (isNaN(valorNumerico)) {
|
||||
@ -88,25 +139,14 @@ export function TransactionForm({ onSuccess, onCancel }: TransactionFormProps) {
|
||||
? Math.abs(valorNumerico)
|
||||
: Math.abs(valorNumerico);
|
||||
|
||||
// Obtém a sessão atual para acessar o token JWT
|
||||
const { data: sessionData, error: sessionError } = await supabase.auth.getSession();
|
||||
|
||||
if (sessionError || !sessionData.session) {
|
||||
console.error('Erro ao obter sessão:', sessionError);
|
||||
toast({
|
||||
title: "Erro de autenticação",
|
||||
description: "Não foi possível validar sua sessão. Tente fazer login novamente.",
|
||||
variant: "destructive"
|
||||
});
|
||||
setIsSubmitting(false);
|
||||
return;
|
||||
}
|
||||
|
||||
// Cria o nome da tabela dinâmica para este usuário
|
||||
const tabelaTransacoes = `transacoes_${userId}`;
|
||||
|
||||
console.log(`Inserindo transação na tabela ${tabelaTransacoes} para o usuário ${userId}`);
|
||||
|
||||
// Obtém a sessão atual para acessar o token JWT - nova solicitação após validação
|
||||
const { data: sessionData } = await supabase.auth.getSession();
|
||||
|
||||
// Insere a transação na tabela específica do usuário
|
||||
const { error } = await supabase
|
||||
.from(tabelaTransacoes as any)
|
||||
@ -124,11 +164,21 @@ export function TransactionForm({ onSuccess, onCancel }: TransactionFormProps) {
|
||||
|
||||
if (error) {
|
||||
console.error('Erro ao salvar transação:', error);
|
||||
|
||||
if (error.message.includes('JWTClaimsSetVerificationException')) {
|
||||
toast({
|
||||
title: "Erro de autenticação",
|
||||
description: "Sua sessão expirou. Por favor, faça login novamente.",
|
||||
variant: "destructive"
|
||||
});
|
||||
navigate('/auth');
|
||||
} else {
|
||||
toast({
|
||||
title: "Erro ao salvar",
|
||||
description: `Não foi possível salvar a transação: ${error.message}`,
|
||||
variant: "destructive"
|
||||
});
|
||||
}
|
||||
} else {
|
||||
toast({
|
||||
title: "Transação salva",
|
||||
|
||||
@ -11,18 +11,33 @@ export async function getTransacoes(): Promise<Transaction[]> {
|
||||
return [];
|
||||
}
|
||||
|
||||
try {
|
||||
// Verificar se temos uma sessão válida
|
||||
const { data: sessionData } = await supabase.auth.getSession();
|
||||
if (!sessionData.session) {
|
||||
console.error('Sessão não encontrada');
|
||||
const { data: sessionData, error: sessionError } = await supabase.auth.getSession();
|
||||
if (sessionError) {
|
||||
console.error('Erro ao obter sessão:', sessionError);
|
||||
return [];
|
||||
}
|
||||
|
||||
if (!sessionData.session) {
|
||||
console.log('Sessão não encontrada, tentando atualizar...');
|
||||
|
||||
// Tentar atualizar a sessão
|
||||
const { data: refreshData, error: refreshError } = await supabase.auth.refreshSession();
|
||||
|
||||
if (refreshError || !refreshData.session) {
|
||||
console.error('Não foi possível atualizar a sessão:', refreshError);
|
||||
return [];
|
||||
}
|
||||
|
||||
console.log('Sessão atualizada com sucesso');
|
||||
}
|
||||
|
||||
const tabelaTransacoes = `transacoes_${userId}`;
|
||||
|
||||
console.log(`Consultando tabela: ${tabelaTransacoes}`);
|
||||
|
||||
// Usando .rpc para chamar uma função SQL ou .from() com cast para any para contornar a checagem de tipos
|
||||
// Usando .from() com cast para any para contornar a checagem de tipos
|
||||
const { data, error } = await supabase
|
||||
.from(tabelaTransacoes as any)
|
||||
.select('*')
|
||||
@ -47,6 +62,10 @@ export async function getTransacoes(): Promise<Transaction[]> {
|
||||
tipo: item.tipo?.toLowerCase() || 'despesa',
|
||||
categoria: item.categoria || 'Outros'
|
||||
}));
|
||||
} catch (error) {
|
||||
console.error('Erro ao buscar transações:', error);
|
||||
return [];
|
||||
}
|
||||
}
|
||||
|
||||
export async function getTransactionSummary() {
|
||||
@ -58,13 +77,28 @@ export async function getTransactionSummary() {
|
||||
return { receitas: 0, despesas: 0, saldo: 0 };
|
||||
}
|
||||
|
||||
try {
|
||||
// Verificar se temos uma sessão válida
|
||||
const { data: sessionData } = await supabase.auth.getSession();
|
||||
if (!sessionData.session) {
|
||||
console.error('Sessão não encontrada');
|
||||
const { data: sessionData, error: sessionError } = await supabase.auth.getSession();
|
||||
if (sessionError) {
|
||||
console.error('Erro ao obter sessão:', sessionError);
|
||||
return { receitas: 0, despesas: 0, saldo: 0 };
|
||||
}
|
||||
|
||||
if (!sessionData.session) {
|
||||
console.log('Sessão não encontrada, tentando atualizar...');
|
||||
|
||||
// Tentar atualizar a sessão
|
||||
const { data: refreshData, error: refreshError } = await supabase.auth.refreshSession();
|
||||
|
||||
if (refreshError || !refreshData.session) {
|
||||
console.error('Não foi possível atualizar a sessão:', refreshError);
|
||||
return { receitas: 0, despesas: 0, saldo: 0 };
|
||||
}
|
||||
|
||||
console.log('Sessão atualizada com sucesso');
|
||||
}
|
||||
|
||||
const tabelaTransacoes = `transacoes_${userId}`;
|
||||
|
||||
const { data, error } = await supabase
|
||||
@ -94,6 +128,10 @@ export async function getTransactionSummary() {
|
||||
|
||||
console.log("Resumo calculado:", resultado);
|
||||
return resultado;
|
||||
} catch (error) {
|
||||
console.error('Erro ao buscar resumo das transações:', error);
|
||||
return { receitas: 0, despesas: 0, saldo: 0 };
|
||||
}
|
||||
}
|
||||
|
||||
export async function getCategorySummary() {
|
||||
@ -105,6 +143,28 @@ export async function getCategorySummary() {
|
||||
return [];
|
||||
}
|
||||
|
||||
try {
|
||||
// Verificar se temos uma sessão válida
|
||||
const { data: sessionData, error: sessionError } = await supabase.auth.getSession();
|
||||
if (sessionError) {
|
||||
console.error('Erro ao obter sessão:', sessionError);
|
||||
return [];
|
||||
}
|
||||
|
||||
if (!sessionData.session) {
|
||||
console.log('Sessão não encontrada, tentando atualizar...');
|
||||
|
||||
// Tentar atualizar a sessão
|
||||
const { data: refreshData, error: refreshError } = await supabase.auth.refreshSession();
|
||||
|
||||
if (refreshError || !refreshData.session) {
|
||||
console.error('Não foi possível atualizar a sessão:', refreshError);
|
||||
return [];
|
||||
}
|
||||
|
||||
console.log('Sessão atualizada com sucesso');
|
||||
}
|
||||
|
||||
const tabelaTransacoes = `transacoes_${userId}`;
|
||||
|
||||
// Buscar todas as transações que são despesas, independente da capitalização
|
||||
@ -154,6 +214,10 @@ export async function getCategorySummary() {
|
||||
|
||||
console.log("Resumo de categorias calculado:", resultado);
|
||||
return resultado;
|
||||
} catch (error) {
|
||||
console.error('Erro ao buscar resumo de categorias:', error);
|
||||
return [];
|
||||
}
|
||||
}
|
||||
|
||||
export async function getMonthlyData() {
|
||||
@ -165,6 +229,28 @@ export async function getMonthlyData() {
|
||||
return [];
|
||||
}
|
||||
|
||||
try {
|
||||
// Verificar se temos uma sessão válida
|
||||
const { data: sessionData, error: sessionError } = await supabase.auth.getSession();
|
||||
if (sessionError) {
|
||||
console.error('Erro ao obter sessão:', sessionError);
|
||||
return [];
|
||||
}
|
||||
|
||||
if (!sessionData.session) {
|
||||
console.log('Sessão não encontrada, tentando atualizar...');
|
||||
|
||||
// Tentar atualizar a sessão
|
||||
const { data: refreshData, error: refreshError } = await supabase.auth.refreshSession();
|
||||
|
||||
if (refreshError || !refreshData.session) {
|
||||
console.error('Não foi possível atualizar a sessão:', refreshError);
|
||||
return [];
|
||||
}
|
||||
|
||||
console.log('Sessão atualizada com sucesso');
|
||||
}
|
||||
|
||||
const tabelaTransacoes = `transacoes_${userId}`;
|
||||
|
||||
const { data, error } = await supabase
|
||||
@ -210,4 +296,8 @@ export async function getMonthlyData() {
|
||||
|
||||
console.log("Dados mensais calculados:", resultado);
|
||||
return resultado;
|
||||
} catch (error) {
|
||||
console.error('Erro ao buscar dados mensais:', error);
|
||||
return [];
|
||||
}
|
||||
}
|
||||
|
||||
Loading…
Reference in New Issue
Block a user