#28 fix(login): handle bad credentials.
This commit is contained in:
@@ -1,26 +1,71 @@
|
|||||||
<template>
|
<template>
|
||||||
<h1>Login</h1>
|
<h1>Login</h1>
|
||||||
<v-form @submit.prevent="login">
|
|
||||||
<v-text-field v-model="username" label="Usuario" required />
|
<v-form ref="loginForm" @submit.prevent="onSubmit">
|
||||||
<v-text-field v-model="password" label="Contraseña" type="password" required />
|
<v-text-field
|
||||||
<v-btn type="submit">Entrar</v-btn>
|
v-model="username"
|
||||||
<v-alert v-if="error" type="error">{{ error }}</v-alert>
|
label="Usuario"
|
||||||
|
:rules="[requiredRule]"
|
||||||
|
required
|
||||||
|
/>
|
||||||
|
<v-text-field
|
||||||
|
v-model="password"
|
||||||
|
label="Contraseña"
|
||||||
|
type="password"
|
||||||
|
:rules="[requiredRule]"
|
||||||
|
required
|
||||||
|
/>
|
||||||
|
|
||||||
|
<v-btn type="submit" color="primary">Entrar</v-btn>
|
||||||
|
|
||||||
|
<v-alert v-if="error" type="error" class="mt-2">{{ error }}</v-alert>
|
||||||
</v-form>
|
</v-form>
|
||||||
</template>
|
</template>
|
||||||
<script setup>
|
|
||||||
import { ref } from 'vue';
|
<script>
|
||||||
import AuthService from '@/services/auth';
|
import AuthService from '@/services/auth';
|
||||||
import { inject } from 'vue';
|
|
||||||
|
|
||||||
const username = ref('');
|
export default {
|
||||||
const password = ref('');
|
name: 'DonConfiao',
|
||||||
const error = ref('');
|
|
||||||
|
|
||||||
async function login() {
|
data() {
|
||||||
try {
|
return {
|
||||||
await AuthService.login({ username: username.value, password: password.value });
|
username: '',
|
||||||
} catch (e) {
|
password: '',
|
||||||
error.value = e.message;
|
error: '',
|
||||||
}
|
};
|
||||||
}
|
},
|
||||||
|
|
||||||
|
methods: {
|
||||||
|
requiredRule(value) {
|
||||||
|
return !!value || 'Este campo es obligatorio';
|
||||||
|
},
|
||||||
|
|
||||||
|
async onSubmit() {
|
||||||
|
this.error = '';
|
||||||
|
|
||||||
|
const form = this.$refs.loginForm;
|
||||||
|
const isValid = await form.validate();
|
||||||
|
|
||||||
|
if (!isValid) return;
|
||||||
|
|
||||||
|
if (!this.username || !this.password) {
|
||||||
|
this.error = 'Usuario y contraseña son obligatorios';
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
try {
|
||||||
|
await AuthService.login({
|
||||||
|
username: this.username,
|
||||||
|
password: this.password,
|
||||||
|
});
|
||||||
|
this.$router.push({ path: '/' });
|
||||||
|
} catch (e) {
|
||||||
|
// Si el servicio devuelve un error (ej. 401) lo convertimos en excepción
|
||||||
|
const msg = e?.response?.data?.message ?? e.message;
|
||||||
|
this.error = msg ?? 'Error al iniciar sesión';
|
||||||
|
}
|
||||||
|
},
|
||||||
|
},
|
||||||
|
};
|
||||||
</script>
|
</script>
|
||||||
|
|||||||
@@ -2,19 +2,34 @@ class AuthService {
|
|||||||
static TOKEN_KEY = 'access_token';
|
static TOKEN_KEY = 'access_token';
|
||||||
static REFRESH_KEY = 'refresh_token';
|
static REFRESH_KEY = 'refresh_token';
|
||||||
|
|
||||||
static login(credentials) {
|
static async login(credentials) {
|
||||||
const url = `${import.meta.env.VITE_DJANGO_BASE_URL}/api/token/`;
|
const url = `${import.meta.env.VITE_DJANGO_BASE_URL}/api/token/`;
|
||||||
return fetch(url, {
|
|
||||||
|
const resp = await fetch(url, {
|
||||||
method: 'POST',
|
method: 'POST',
|
||||||
headers: { 'Content-Type': 'application/json' },
|
headers: { 'Content-Type': 'application/json' },
|
||||||
body: JSON.stringify(credentials),
|
body: JSON.stringify(credentials),
|
||||||
})
|
});
|
||||||
.then(r => r.json())
|
|
||||||
.then(data => {
|
if (!resp.ok) {
|
||||||
localStorage.setItem(this.TOKEN_KEY, data.access);
|
let errMsg = resp.statusText;
|
||||||
localStorage.setItem(this.REFRESH_KEY, data.refresh);
|
try {
|
||||||
return data;
|
const errData = await resp.json();
|
||||||
});
|
errMsg = errData?.detail ?? errData?.message ?? errMsg;
|
||||||
|
} catch (_) {
|
||||||
|
|
||||||
|
}
|
||||||
|
throw new Error(errMsg);
|
||||||
|
}
|
||||||
|
|
||||||
|
const data = await resp.json();
|
||||||
|
|
||||||
|
if (data.access && data.refresh) {
|
||||||
|
localStorage.setItem(this.TOKEN_KEY, data.access);
|
||||||
|
localStorage.setItem(this.REFRESH_KEY, data.refresh);
|
||||||
|
}
|
||||||
|
|
||||||
|
return data;
|
||||||
}
|
}
|
||||||
|
|
||||||
static getAccessToken() {
|
static getAccessToken() {
|
||||||
@@ -35,6 +50,12 @@ class AuthService {
|
|||||||
headers: { 'Content-Type': 'application/json' },
|
headers: { 'Content-Type': 'application/json' },
|
||||||
body: JSON.stringify({ refresh }),
|
body: JSON.stringify({ refresh }),
|
||||||
});
|
});
|
||||||
|
|
||||||
|
if (!resp.ok) {
|
||||||
|
const errData = await resp.json().catch(() => ({}));
|
||||||
|
throw new Error(errData?.detail ?? resp.statusText);
|
||||||
|
}
|
||||||
|
|
||||||
const data = await resp.json();
|
const data = await resp.json();
|
||||||
localStorage.setItem(this.TOKEN_KEY, data.access);
|
localStorage.setItem(this.TOKEN_KEY, data.access);
|
||||||
return data.access;
|
return data.access;
|
||||||
|
|||||||
Reference in New Issue
Block a user