bitnami/sealed-secrets

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

Offen

#596 geöffnet am 28.05.2021

Auf GitHub ansehenContributor Guide
 (0 Kommentare) (1 Reaktion) (0 zugewiesene Personen)Go (771 Forks)auto 404
backlogbuggood first issue

Repository-Metriken

Stars
 (9.222 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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.

Contributor Guide

Research-Richtung
Durchsuchen Sie die Codebasis, um zu finden, wo das Flag insecure skip tls verify behandelt wird. Derzeit gilt es nur für Kubernetes API Verbindungen. Erweitern Sie es, um die TLS Überprüfung auch beim Abrufen des Zertifikats von der HTTPS URL zu überspringen. Suchen Sie nach der HTTP Client Erstellung und fügen Sie die unsichere Überspring Option basierend auf diesem Flag hinzu.
Tech Stack
go
Domain
clisecurity
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Gokubeseal usage
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75