Màj de PyJWT

[christophehenry]

🌮 Objectif

Mise à jour de la dépendance PyJWT.

J'ai fait la màj en 2 étapes : d'abord passage à la version 2.0.0 pour voir ce qui casse. Puis passage à la dernière version. 2 breaking changes notables ici : jwt.encode retournes un str et plus un bytes ce qui a légèrement modifié 2 tests. La signature de jwt.decode change avec la paramètre algorithm qui devient algorithms et prend une liste.

Pas d'autres changements notables pour nous. Le changelog est là.

[gitguardian]

⚠️ GitGuardian has uncovered 1 secret following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

🔎 Detected hardcoded secret in your pull request

GitGuardian id	Secret	Commit	Filename
21882258	Generic High Entropy Secret	90024ac	aidants_connect_web/views/id_provider.py	View secret

🛠 Guidelines to remediate hardcoded secrets

1. Revoke and rotate the secret.

2. If possible, rewrite git history with git commit --amend and git push --force.

---

<sup>🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

Our GitHub checks need improvements? Share your feedbacks!</sup>

[christophehenry]

Hmm… Étrange… J'avais jamais remarqué… @mrjmad @Anna-Livia pourquoi il est codé en dur ce refresh token ? 🤔

[mrjmad]

je n'avais jamais fait attention, a mon avis ça vaut le coup de à minima le passer en venv et changer sa valeur.

a etudier le fait aussi qu'il soit randomiser dans une futur amélioration ( c'est que un token de refresh mais quand même )

[tut-tuuut]

Vu. Compte tenu du token qui a été exposé, j'aimerais que ça parte en prod très vite.

[christophehenry]

Je désactive le check GitGardian. Il va rester rouge tant que j'ai pas réécrit le commit qui a exposé le token. Sauf qu'il est vieux ce commit, je vais pas réécrire aussi loin dans le passé.

[tut-tuuut]

Tiens, c'est volontaire ça ?

[christophehenry]

Voui, je pensais l'avoir commit dans une autre PR. En fait, --check fail au lieu de modifier le fichier. Et je me suis rendu compte que c'était plus pratique que isort fail pas et modifie le fichier du coup c'est pre-commit qui te stop et t'empêche de commit sans prendre en compte les fix d'isort. Les autres check de pre-commit fonctionnent pareil, en particulier black.