verdnatura/salix
verdnatura
/
salix
3
5
Fork 0
Code Issues Pull Requests 64 Releases Wiki Activity

#6264 - renewToken #1824

Merged
jsegarra merged 36 commits from 6264-renewToken into dev 2023-12-20 09:22:29 +00:00
Conversation 0 Commits 36 Files Changed 11 +7 -10
3 changed files with 7 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 8d29850917 - Show all commits

3
back/methods/vn-user/renew-token.js
View File

@ -29,7 +29,8 @@ module.exports = Self => {
// Check if current token is valid
const isValid = await validateToken(token);
if (isValid) throw new UserError(`The renew period has not been exceeded`, 'periodNotExceeded');
if (isValid)
jsegarra marked this conversation as resolved Outdated
juan commented 2023-12-11 08:54:07 +00:00
Outdated
Review
Copy Link

El método validateToken solo se llama desde aquí en toda la aplicación, puede eliminarse y poner su código directamente aquí.

El método validateToken solo se llama desde aquí en toda la aplicación, puede eliminarse y poner su código directamente aquí.
jsegarra commented 2023-12-12 10:58:53 +00:00
Outdated
Review
Copy Link

Era la idea, pero no la seguí porque había un remoteMethod de validateToken y asumí que si existía seria por algo y había que aprovecharlo.

Consideras que deberíamos mover la lógica de ese método a este y eliminarlo?

Era la idea, pero no la seguí porque había un remoteMethod de validateToken y asumí que si existía seria por algo y había que aprovecharlo. Consideras que deberíamos mover la lógica de ese método a este y eliminarlo?
jgallego commented 2023-12-13 06:30:37 +00:00
Outdated
Review
Copy Link

Si ves que sólo se llama desde aquí sí.

Si ves que sólo se llama desde aquí sí.
👍 1
return token;
const {courtesyTime} = await models.AccessTokenConfig.findOne({fields: ['courtesyTime']});
jsegarra marked this conversation as resolved Outdated
juan commented 2023-12-11 08:53:17 +00:00
Outdated
Review
Copy Link

Para que se selecciona renewPeriod si luego no se utiliza?

Para que se selecciona `renewPeriod` si luego no se utiliza?
😕 1
jsegarra commented 2023-12-12 10:56:52 +00:00
Outdated
Review
Copy Link

Oh, tienes razón la línea venia heredada de otro sitio

Oh, tienes razón la línea venia heredada de otro sitio

10
back/methods/vn-user/specs/renew-token.spec.js
View File

@ -37,13 +37,13 @@ describe('Renew Token', () => {
it('NOT should renew', async() => {
let error;
try {
await models.VnUser.renewToken(ctx);
const response = await models.VnUser.renewToken(ctx);
expect(error).toBeUnDefined();
expect(error.statusCode).toBe(200);
expect(response.token.id).toEqual(ctx.req.accessToken.id);
} catch (e) {
error = e;
}
expect(error).toBeDefined();
expect(error.statusCode).toBe(400);
expect(error.message).toEqual('The renew period has not been exceeded');
});
});

4
front/core/services/token.js
View File

@ -103,10 +103,6 @@ export default class Token {
const token = res.data;
this.set(token.id, now, token.ttl, this.remember);
})
.catch(res => {
if (res.data?.error?.code !== 'periodNotExceeded')
throw res;
})
.finally(() => {
this.checking = false;
});