palantir / palantir/python-compute-module
[bug] `RefreshingOauthToken` should not accept a `refresh_interval` > token expiry
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 15
- Forks
- 11
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
If a user specifies an arbitrarily large refresh_interval for RefreshingOauthToken, that can lead to an expired token being used. This value should not be any greater than the expires_in value returned from the oauth call.
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne in compute_modules/auth/third_party.py bei der RefreshingOauthToken-Implementierung, auf die im Issue verwiesen wird. Untersuche, wie refresh_interval und der expires_in-Wert der OAuth-Antwort behandelt werden, und verifiziere anschließend das abgeschlossene Verhalten mit den relevanten Tests des Repositorys. Erledigt ist es, wenn ein beliebig großes refresh_interval nicht dazu führen kann, dass ein abgelaufenes Token verwendet wird.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- authentication
- Issue-Typ
- Bug
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Ruhig
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 76/100