bitnami/sealed-secrets

--insecure-skip-tls-verify works only for K8s API connections

Ouverte

#596 ouverte le 28 mai 2021

Voir sur GitHubGuide contributeur
 (0 commentaire) (1 réaction) (0 personne assignée)Go (771 forks)auto 404
backlogbuggood first issue

Métriques du dépôt

Stars
 (9 222 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

I am fetching a cert from the sealed secrets like this:

 kubeseal --raw --insecure-skip-tls-verify=true --from-file=foo.secret --cert https://sealed-secrets.my-ingress.com/v1/cert.pem --namespace team-a --name team-a-foo

The ingresses certificate is self signed. I expected --insecure-skip-tls-verify to ignore tls verification, but looking at the code of sealed-secrets, this flag is never used inside this project and is probably only passed down to K8s Go client.

I believe it should be an easy fix though - should this flag be used for both use cases? If yes, I can try to submit a quick fix.

Guide contributeur

Direction de recherche
Inspectez le codebase pour trouver où le drapeau insecure skip tls verify est traité. Actuellement, il ne s'applique qu'aux connexions à l'API Kubernetes. Étendez le pour également sauter la vérification TLS lors de la récupération du certificat depuis l'URL HTTPS. Recherchez la création du client HTTP et ajoutez l'option de saut non sécurisé basée sur ce drapeau.
Stack technique
go
Domaine
clisecurity
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.
2
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
Gokubeseal usage
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75