alibaba/Sentinel

sentinel-apache-dubbo-adapter 在异步模式下永远取不到异常

Open

#2 322 ouverte le 9 août 2021

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)Java (8 150 forks)batch import
area/integrationsgood first issue

Métriques du dépôt

Stars
 (23 109 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

代码：

https://github.com/alibaba/Sentinel/blob/4498de480f368d6bf63afbfda42797172c89573b/sentinel-adapter/sentinel-dubbo-adapter/src/main/java/com/alibaba/csp/sentinel/adapter/dubbo/SentinelDubboProviderFilter.java#L73

在 dubbo 2.7.5 后，所有Filter均支持异步操作，返回的 Result 对象通常是 AsyncRpcResult，对 AsyncRpcResult 调用hasException() 时如果异步任务未完成，则使用默认的返回值，永远不包含异常。

https://github.com/apache/dubbo/blob/9c49efeacfd87d2d4409fb000cebd58e1114ec8a/dubbo-rpc/dubbo-rpc-api/src/main/java/org/apache/dubbo/rpc/AsyncRpcResult.java#L143-L155

Guide contributeur

Direction de recherche
Le problème décrit qu'en mode asynchrone, le filtre Dubbo de Sentinel ne parvient pas à attraper les exceptions car il appelle hasException() sur AsyncRpcResult avant la fin. Pour corriger, examinez la ligne 73 du fichier SentinelDubboProviderFilter.java lié et AsyncRpcResult.java dans Dubbo. La solution implique de vérifier si le résultat est un AsyncRpcResult, puis de gérer la récupération des exceptions différemment, éventuellement en attachant un callback ou en vérifiant après la fin. Consultez les commentaires et les réponses du mainteneur pour des conseils supplémentaires.
Stack technique
java
Domaine
backendapi
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Java
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70