Add com.twitter.finagle.zipkin.core.Endpoint.ipv6

Métriques du dépôt

Stars

8 864 stars (8 864 stars)

Métriques de merge PR

Aucune PR mergée en 30 j (Aucune PR mergée en 30 j)

Description

Guide contributeur

Direction de recherche

Le problème consiste à ajouter la prise en charge d'IPv6 à la classe Endpoint dans l'intégration Zipkin de Finagle. Le changement clé se situe dans la classe com.twitter.finagle.zipkin.core.Endpoint, qui ne prend actuellement en charge que l'IPv4. La spécification de l'API Zipkin (https://github.com/openzipkin/zipkin api/blob/master/thrift/zipkinCore.thrift#L276) indique que les endpoints peuvent désormais inclure optionnellement un champ ipv6 en tant qu'adresse de 16 octets. Pour implémenter cela, vous devrez modifier la classe Scala Endpoint pour ajouter un champ ipv6, mettre à jour sa sérialisation/désérialisation pour gérer le nouveau champ et garantir la rétrocompatibilité avec les endpoints IPv4 uniquement. Le problème mentionne également un contributeur (@sveinnfannar) qui pourrait fournir plus de contexte. Examinez la base de code existante pour comprendre l'implémentation actuelle d'Endpoint et la liaison Thrift, puis ajoutez la prise en charge d'ipv6 en conséquence.

Stack technique

scala

Domaine

backendobservability

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.

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.

Ancienne

ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.

Claire

Prérequis

Scala knowledgeFamiliarity with ThriftUnderstanding of Zipkin endpoint

Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.

40