verdnatura/salix
verdnatura
/
salix
3
4
Fork 0
Code Issues Pull Requests 59 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 +4 -6
3 changed files with 4 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit d4cd23853f - Show all commits

6
back/methods/vn-user/is-token-valid.js
View File

@ -2,10 +2,8 @@ const tokenConfig = require('./token-config');
module.exports = async token => {
jsegarra marked this conversation as resolved Outdated
alexm commented 2023-11-06 09:07:48 +00:00
Outdated
Review
Copy Link

Creo que se seria mas correcto usar vnNew()

Mas info: https://wiki.verdnatura.es/index.php/Fechas

Creo que se seria mas correcto usar vnNew() Mas info: https://wiki.verdnatura.es/index.php/Fechas
jsegarra commented 2023-11-09 09:05:01 +00:00
Outdated
Review
Copy Link

Okey, tomo nota.
Lo extraje de lo que hay actualmente de renew-token

Okey, tomo nota. Lo extraje de lo que hay actualmente de renew-token
const accessTokenConfig = await tokenConfig();
let now = Date.now();
if (Date?.vnNow !== undefined)
now = Date.vnNow();
const now = Date.now();
const differenceMilliseconds = now - token.created;
const differenceSeconds = Math.floor(differenceMilliseconds / 1000);
return differenceSeconds > accessTokenConfig.renewPeriod - accessTokenConfig.courtesyTime;
return differenceSeconds < accessTokenConfig.renewPeriod - accessTokenConfig.courtesyTime;
};

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

@ -29,7 +29,7 @@ module.exports = Self => {
// Check if current token is valid
const isValid = await Self.validateToken(token);
if (!isValid) throw new UserError(`The renew period has not been exceeded`, 'periodNotExceeded');
if (isValid) throw new UserError(`The renew period has not been exceeded`, 'periodNotExceeded');
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
const {courtesyTime} = await tokenConfig();
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

2
back/methods/vn-user/specs/validate-token.spec.js
View File

@ -29,13 +29,13 @@ describe('Validate Token', () => {
});
it('Token is not expired', async() => {
jasmine.clock().mockDate(new Date(startingTime + 21600000));
const isValid = await models.VnUser.validateToken(ctx.req.accessToken);
expect(isValid).toBeTrue();
});
it('Token is expired', async() => {
jasmine.clock().mockDate(new Date(startingTime + 21600000));
const isValid = await models.VnUser.validateToken(ctx.req.accessToken);
expect(isValid).toBeFalse();