Add kubernetes client back

_backup/solver_local.go

@@ -27,7 +27,7 @@ type LocalDNSProviderSolver struct {
 // within a single webhook deployment**.
 // For example, `cloudflare` may be used as the name of a solver.
 func (p *LocalDNSProviderSolver) Name() string {
-	return providerName
+	return ProviderName
 }
 
 // Present is responsible for actually presenting the DNS record with the

cmd/buildversion.go

@@ -10,7 +10,6 @@ import (
 )
 
 const (
-	version     = "0.0.4-alpha"
 	chartfile   = "./deploy/sthome-webhook/Chart.yaml"
 	valuesfile  = "./deploy/sthome-webhook/values.yaml"
 	tagprefix   = "  tag: "
@@ -38,9 +37,11 @@ func main() {
 	// Generate a timestamp.
 	buildTime = time.Now().Format("20060102-1504")
 
-	// Load the count from the 3rd line of the file
-	// It's a string so we need to convert to integer
-	// Then increment it by 1
+	// deconstruct version line and remove build increment part
+	verparts := strings.Split(vLines[0], "-")
+	versionmins := strings.Split(verparts[1], ".")
+	version := verparts[0] + "-" + versionmins[0]
+
 	bNum, _ := strconv.Atoi(vLines[2])
 	bNum++
 	longversion = version + "." + fmt.Sprint(bNum)

deploy/sthome-webhook/Chart.yaml

@@ -1,5 +1,5 @@
 apiVersion: v1
-appVersion: v0.0.4-alpha.112
+appVersion: v0.0.5-alpha.1
 description: Cert-Manager webhook for sthome
 name: sthome-webhook
-version: 0.0.4-alpha.112
+version: 0.0.5-alpha.1

deploy/sthome-webhook/values.yaml

@@ -31,7 +31,7 @@ clusterIssuer:
 image:
   repository: stuurmcp/cert-manager-webhook-sthome
   #repository: wstat.sthome.net:5000/cert-manager-webhook-sthome
-  tag: 0.0.4-alpha.112
+  tag: 0.0.5-alpha.1
   #pullPolicy should be IfNotPresent. Set to Always for testing purposes
   pullPolicy: IfNotPresent
 

go.mod

@@ -12,12 +12,12 @@ require (
 	k8s.io/client-go v0.29.0
 )
 
-require (
-	k8s.io/apimachinery v0.29.0
-	k8s.io/klog/v2 v2.110.1
-)
+require k8s.io/klog/v2 v2.110.1
 
-require github.com/pmezard/go-difflib v1.0.1-0.20181226105442-5d4384ee4fb2 // indirect
+require (
+	github.com/pmezard/go-difflib v1.0.1-0.20181226105442-5d4384ee4fb2 // indirect
+	k8s.io/apimachinery v0.29.0 // indirect
+)
 
 require (
 	github.com/NYTimes/gziphandler v1.1.1 // indirect

main.go

@@ -2,11 +2,17 @@ package main
 
 import (
 	_ "embed"
+	"fmt"
+	"os"
 	"strings"
 
-	"os"
+	"k8s.io/client-go/kubernetes"
+	"k8s.io/client-go/rest"
+	"k8s.io/klog/v2"
 
+	"github.com/cert-manager/cert-manager/pkg/acme/webhook/apis/acme/v1alpha1"
 	"github.com/cert-manager/cert-manager/pkg/acme/webhook/cmd"
+	"github.com/cert-manager/cert-manager/pkg/issuer/acme/dns/util"
 
 	dns "github.com/stuurmcp/cert-manager-webhook-sthome/pkg/dns"
 )
@@ -28,3 +34,83 @@ func main() {
 		&dns.SthomeSolver{},
 	)
 }
+
+type LocalDNSProviderSolver struct {
+	client kubernetes.Interface
+}
+
+func (p *LocalDNSProviderSolver) Name() string {
+	return dns.ProviderName
+}
+
+func (loc *LocalDNSProviderSolver) Present(ch *v1alpha1.ChallengeRequest) error {
+	cfg, err := dns.LoadConfig(ch.Config)
+	if err != nil {
+		return err
+	}
+	klog.Infof("CZ: Presenting record for %s, type: %s, uid: %s, key: %s, ns: %s, fqdn: %s, zone: %s, allowambcred: %t, cfg.secret: %s, cfg.email: %s, cfg.allowz: %s",
+		ch.DNSName,
+		ch.UID,
+		ch.Type,
+		ch.Key,
+		ch.ResourceNamespace,
+		ch.ResolvedFQDN,
+		ch.ResolvedZone,
+		ch.AllowAmbientCredentials,
+		cfg.APIKeySecretRef.Name,
+		cfg.Email,
+		strings.Join(cfg.AllowedZones, ","),
+	)
+	// TODO: convert shell script to golang
+	localip := dns.GetOutboundIP(dns.Dnsserver_net)
+	success, _ := dns.Execute(
+		dns.Shell,
+		dns.AcmeAuthCmd,
+		"set",
+		ch.DNSName,
+		ch.ResolvedFQDN,
+		ch.Key,
+		"-l",
+		localip,
+		"-v",
+	)
+	klog.Infof("Execute set TXT returned success: %t", success)
+	return nil
+}
+
+func (loc *LocalDNSProviderSolver) CleanUp(ch *v1alpha1.ChallengeRequest) error {
+	//domainName := extractDomainName(ch.ResolvedZone)
+	// TODO: add code that deletes a record from the DNS provider's console
+	localip := dns.GetOutboundIP(dns.Dnsserver_net)
+	success, _ := dns.Execute(
+		dns.Shell,
+		dns.AcmeAuthCmd,
+		"unset",
+		ch.DNSName,
+		ch.ResolvedFQDN,
+		ch.Key,
+		"-l",
+		localip,
+		"-v",
+	)
+	klog.Infof("Execute unset TXT returned success: %t", success)
+	return nil
+}
+func (loc *LocalDNSProviderSolver) Initialize(kubeClientConfig *rest.Config, stopCh <-chan struct{}) error {
+	cl, err := kubernetes.NewForConfig(kubeClientConfig)
+	if err != nil {
+		return fmt.Errorf("failed to get kubernetes client: %w", err)
+	}
+	loc.client = cl
+	klog.InfoS("CZ: Successfully initialised kubernetes client!")
+	return nil
+}
+
+func extractDomainName(zone string) string {
+	authZone, err := util.FindZoneByFqdn(zone, util.RecursiveNameservers)
+	if err != nil {
+		klog.Errorf("could not get zone by fqdn %v", err)
+		return zone
+	}
+	return util.UnFqdn(authZone)
+}

pkg/dns/config.go

@@ -12,14 +12,14 @@ const (
 	SthomeAccessKeyEnv = "STHOME_ACCESS_KEY"
 	SthomeSecretKeyEnv = "STHOME_SECRET_KEY"
 
-	providerName = "sthome"
-	shell        = "/bin/bash"
-	acmeAuthCmd  = "/acme/acmeauth.sh"
+	ProviderName = "sthome"
+	Shell        = "/bin/bash"
+	AcmeAuthCmd  = "/acme/acmeauth.sh"
 
-	dnsserver_net  = "10.0.0.15"
-	dnsserver_lan  = "192.168.2.1"
-	hostserver_net = "truenas.sthome.net"
-	hostserver_lan = "truenas.sthome.lan"
+	Dnsserver_net  = "10.0.0.15"
+	Dnsserver_lan  = "192.168.2.1"
+	Hostserver_net = "truenas.sthome.net"
+	Hostserver_lan = "truenas.sthome.lan"
 )
 
 // localDNSProviderConfig is a structure that is used to decode into when
@@ -111,7 +111,7 @@ func (cfg LocalDNSProviderConfig) IsAllowedZone(zone string) bool {
 }
 
 // Get preferred outbound ip of this machine
-func getOutboundIP(dest string) string {
+func GetOutboundIP(dest string) string {
 	conn, err := net.Dial("udp", dest+":80")
 	if err != nil {
 		klog.Errorf("net.Dial error: %s", err)

pkg/dns/solver_sthome.go

@@ -29,7 +29,7 @@ func (e *SthomeSolver) Present(ch *acme.ChallengeRequest) error {
 	e.Lock()
 	e.txtRecords[ch.ResolvedFQDN] = ch.Key
 	e.Unlock()
-	cfg, err := loadConfig(ch.Config)
+	cfg, err := LoadConfig(ch.Config)
 	if err != nil {
 		return err
 	}
@@ -47,10 +47,10 @@ func (e *SthomeSolver) Present(ch *acme.ChallengeRequest) error {
 		strings.Join(cfg.AllowedZones, ","),
 	)
 	// TODO: convert shell script to golang
-	localip := getOutboundIP(dnsserver_net)
+	localip := GetOutboundIP(Dnsserver_net)
 	success, _ := Execute(
-		shell,
-		acmeAuthCmd,
+		Shell,
+		AcmeAuthCmd,
 		"set",
 		ch.DNSName,
 		ch.ResolvedFQDN,
@@ -67,10 +67,10 @@ func (e *SthomeSolver) CleanUp(ch *acme.ChallengeRequest) error {
 	e.Lock()
 	delete(e.txtRecords, ch.ResolvedFQDN)
 	e.Unlock()
-	localip := getOutboundIP(dnsserver_net)
+	localip := GetOutboundIP(Dnsserver_net)
 	success, _ := Execute(
-		shell,
-		acmeAuthCmd,
+		Shell,
+		AcmeAuthCmd,
 		"unset",
 		ch.DNSName,
 		ch.ResolvedFQDN,
@@ -101,7 +101,7 @@ func (e *SthomeSolver) Initialize(kubeClientConfig *rest.Config, stopCh <-chan s
 
 func New(port string) webhook.Solver {
 	e := &SthomeSolver{
-		name:       "sthome",
+		name:       ProviderName,
 		txtRecords: make(map[string]string),
 	}
 	e.server = &dns.Server{

pkg/dns/utils.go

@@ -12,7 +12,7 @@ import (
 
 // loadConfig is a small helper function that decodes JSON configuration into
 // the typed config struct.
-func loadConfig(cfgJSON *extapi.JSON) (LocalDNSProviderConfig, error) {
+func LoadConfig(cfgJSON *extapi.JSON) (LocalDNSProviderConfig, error) {
 	cfg := LocalDNSProviderConfig{}
 	// handle the 'base case' where no configuration has been provided
 	if cfgJSON == nil {

pkg/dns/utils_test.go

@@ -1,9 +1,7 @@
 package dns
 
 import (
-	"context"
 	"fmt"
-	"os"
 	"reflect"
 	"testing"
 
@@ -11,8 +9,6 @@ import (
 	//client "github.com/kubernetes-sdk-for-go-101/pkg/client"
 	v1 "k8s.io/api/core/v1"
 	extapi "k8s.io/apiextensions-apiserver/pkg/apis/apiextensions/v1"
-	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
-	testclient "k8s.io/client-go/kubernetes/fake"
 )
 
 func Test_loadConfig(t *testing.T) {
@@ -67,7 +63,7 @@ func Test_loadConfig(t *testing.T) {
 		if test.json == "" {
 			json = nil
 		}
-		config, err := loadConfig(json)
+		config, err := LoadConfig(json)
 		if err != nil {
 			if !test.shouldErr {
 				t.Errorf("got error %v where no error was expected", err)
@@ -81,6 +77,7 @@ func Test_loadConfig(t *testing.T) {
 	}
 }
 
+/*
 func Test_getDomainAPI(t *testing.T) {
 	jsonBothKey := &extapi.JSON{
 		Raw: []byte(`{
@@ -158,42 +155,42 @@ func Test_getDomainAPI(t *testing.T) {
 			shouldErr: false,
 		},
 	}
+		for _, test := range testCases {
+			fakeKubernetesClient := testclient.NewSimpleClientset()
+			pSolver := &main.LocalDNSProviderSolver{
+				client: fakeKubernetesClient,
+			}
 
-	for _, test := range testCases {
-		fakeKubernetesClient := testclient.NewSimpleClientset()
-		pSolver := &LocalDNSProviderSolver{
-			client: fakeKubernetesClient,
-		}
-
-		if test.secret != nil {
-			_, err := pSolver.client.CoreV1().Secrets(test.ch.ResourceNamespace).Create(context.Background(), test.secret, metav1.CreateOptions{})
+			if test.secret != nil {
+				_, err := pSolver.client.CoreV1().Secrets(test.ch.ResourceNamespace).Create(context.Background(), test.secret, metav1.CreateOptions{})
+				if err != nil {
+					t.Errorf("failed to create kubernetes secret")
+				}
+			}
+			for k, v := range test.env {
+				os.Setenv(k, v)
+			}
+			_, err := getDomainAPI(test.ch)
 			if err != nil {
-				t.Errorf("failed to create kubernetes secret")
+				if !test.shouldErr {
+					t.Errorf("got error %v where no error was expected", err)
+				}
+				if err.Error() != test.errMessage {
+					t.Errorf("expected error %s, got %s", test.errMessage, err.Error())
+				}
+			} else if test.shouldErr {
+				t.Errorf("didn't get an error where an error was expected with message %s", test.errMessage)
+			}
+			for k := range test.env {
+				os.Unsetenv(k)
 			}
 		}
-		for k, v := range test.env {
-			os.Setenv(k, v)
-		}
-		_, err := getDomainAPI(test.ch)
-		if err != nil {
-			if !test.shouldErr {
-				t.Errorf("got error %v where no error was expected", err)
-			}
-			if err.Error() != test.errMessage {
-				t.Errorf("expected error %s, got %s", test.errMessage, err.Error())
-			}
-		} else if test.shouldErr {
-			t.Errorf("didn't get an error where an error was expected with message %s", test.errMessage)
-		}
-		for k := range test.env {
-			os.Unsetenv(k)
-		}
-	}
-}
 
+}
+*/
 // dummy, should actually return the API, but here we just return the config
 func getDomainAPI(ch *v1alpha1.ChallengeRequest) (*LocalDNSProviderConfig, error) {
-	config, err := loadConfig(ch.Config)
+	config, err := LoadConfig(ch.Config)
 	if err != nil {
 		return nil, fmt.Errorf("failed to load config: %w", err)
 	}

version.txt

@@ -1,3 +1,3 @@
-0.0.4-alpha.112
-20240406-0925
-112
+0.0.5-alpha.1
+20240406-1230
+1