maelgangloff / scolengo-api

Unofficial Node.js API client of Skolengo EMS v2
https://npmjs.com/package/scolengo-api
GNU General Public License v3.0
25 stars 4 forks source link

Ajout de refreshToken dans la config & fixs JWT #47

Closed NonozgYtb closed 2 months ago

NonozgYtb commented 2 months ago

Description de la demande de fusion

Il y a eu un oubli dans le constructeur. Selon la documentation, on peut passer en paramètre une fonction refreshToken, mais elle n'est jamais ajouté dans la config this.config. Un petit fix également pour le JWT qui dans certains cas ne marche pas. En se basant sur jwt-decode, l'ajout de deux replace semble fonctionner.

Type de changement

Checklist:

Note

Avant de fusionner cette PR, elle devra recevoir l'approbation d'au moins une des personnes suivantes :

vinceh121 commented 2 months ago

En effet les JWT s'appuient sur la variante url-safe base64url, où les slash et plus sont remplacés par des underscores et tirets