spring-attic/spring-cloud-security

OAuth2 intercepter for feign does not re-issue invalid token

Ouverte

#220 ouverte le 2 sept. 2019

Voir sur GitHubGuide contributeur
 (3 commentaires) (8 réactions) (0 personne assignée)Java (250 forks)batch import
enhancementhelp wanted

Métriques du dépôt

Stars
 (525 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Hi,

If we check OAuth2RestTemplate it has an option retryBadAccessTokens which will try one more time to obtain new access token if previous token is invalid. This is useful if, for some reasons, you decide to revoke access token.

In OAuth2FeignRequestInterceptor access token is preserved in client context until it expires. There is no way to make feign to reissue new token until it expire.

The difficulty here that in feign interceptor is not responsible for retry. So what will be correct way to handle this issue? One of the option is to implement a feign retryer which will clean up client context.

If it is correct, do you want me to try to contribute this change?

Guide contributeur

Direction de recherche
L'issue demande un mécanisme de nouvelle tentative pour les jetons OAuth2 invalides dans les intercepteurs Feign. L'auteur suggère d'implémenter un Feign Retryer pour nettoyer le contexte client lors de l'invalidation du jeton. Pour contribuer, examinez l'OAuth2FeignRequestInterceptor actuel et comprenez comment les jetons sont stockés. Envisagez d'ajouter un retryer qui capture les exceptions et actualise le jeton. Comme le dépôt est archivé, vérifiez s'il existe des projets successeurs ou notez que l'issue devra peut être être traitée dans une autre bibliothèque.
Stack technique
javaspring
Domaine
backendsecurity
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaSpringGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65