helm/helm

Helm session HTTP header

Open

#13 182 ouverte le 16 juil. 2024

Voir sur GitHub
 (18 commentaires) (0 réactions) (0 assignés)Go (7 602 forks)batch import
Stalefeaturehelp wanted

Métriques du dépôt

Stars
 (29 815 stars)
Métriques de merge PR
 (Merge moyen 40j 17h) (48 PRs mergées en 30 j)

Description

Hey there!

kubectl has this nice kubectl-session header when doing multiple requests to the Kubernetes API, which can help for many things in the middle (e.g. stickiness based on HTTP headers, audit, etc.) by giving a unique id related to a single command from the end user.

From what I've seen, Helm doesn't have a session header. Is there any way to know from a MITM perspective which requests are grouped together? If not, how would you feel about adding a helm-session header? :)

Guide contributeur

Direction de recherche
Regardez comment kubectl implémente l'en tête de session (kubectl session) pour référence. Dans Helm, localisez la configuration du client HTTP, probablement dans `pkg/action` ou `internal/clients`. Identifiez où les requêtes sont faites vers l'API Kubernetes. L'ID de session doit être généré par commande et attaché à toutes les requêtes. Vérifiez les discussions existantes dans les commentaires de l'issue pour tout détail d'implémentation proposé. Le changement nécessite d'ajouter un en tête à la requête HTTP, éventuellement dans un wrapper de transport ou un constructeur de client.
Stack technique
go
Domaine
backendinfrastructure
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.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GoHTTP
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60