From 55964f3790449c705a4fb176921730443daff5bd Mon Sep 17 00:00:00 2001
From: nhyatt <nhyatt@smoothnet.org>
Date: Fri, 24 Mar 2023 09:46:15 -0500
Subject: [PATCH] move admin function to specefic admin api

---
 cmd/webhook/httpServer.go          |  4 +++-
 cmd/webhook/httpServerTemplates.go | 30 ++++++++++++++++++++++++------
 2 files changed, 27 insertions(+), 7 deletions(-)

diff --git a/cmd/webhook/httpServer.go b/cmd/webhook/httpServer.go
index 05bd7a1..7d5994d 100644
--- a/cmd/webhook/httpServer.go
+++ b/cmd/webhook/httpServer.go
@@ -101,10 +101,12 @@ func webServe() http.HandlerFunc {
 			msg := fmt.Sprintf("incorrect method: got request type %s, expected request type %s", r.Method, http.MethodPost)
 			log.Printf("[DEBUG] %s", msg)
 			tmpltError(w, http.StatusMethodNotAllowed, msg)
+		case r.URL.Path == "/api/v1/admin":
+			tmpltAdminToggle(w, r.URL.Query())
 		case r.URL.Path == "/healthcheck":
 			tmpltHealthCheck(w)
 		case r.URL.Path == "/":
-			tmpltWebRoot(w, r.URL.Query())
+			tmpltWebRoot(w)
 		default:
 			msg := fmt.Sprintf("Unable to locate requested path: '%s'", r.URL.Path)
 			log.Printf("[DEBUG] %s", msg)
diff --git a/cmd/webhook/httpServerTemplates.go b/cmd/webhook/httpServerTemplates.go
index 66a969e..be4bc70 100644
--- a/cmd/webhook/httpServerTemplates.go
+++ b/cmd/webhook/httpServerTemplates.go
@@ -54,7 +54,7 @@ func tmpltHealthCheck(w http.ResponseWriter) {
 	w.Write(output) //nolint:errcheck
 }
 
-func tmpltWebRoot(w http.ResponseWriter, urlPrams url.Values) {
+func tmpltWebRoot(w http.ResponseWriter) {
 	o := struct {
 		Application string `json:"application" yaml:"application"`
 		Description string `json:"description" yaml:"description"`
@@ -66,17 +66,35 @@ func tmpltWebRoot(w http.ResponseWriter, urlPrams url.Values) {
 	}
 	w.Header().Add(cT, cTjson)
 
+	output, err := json.MarshalIndent(o, "", "  ")
+	if err != nil {
+		log.Printf(marshalErrorMsg, err)
+	}
+	w.Write(output) //nolint:errcheck
+}
+
+func tmpltAdminToggle(w http.ResponseWriter, urlPrams url.Values) {
+	o := struct {
+		Application string `json:"application" yaml:"application"`
+		Description string `json:"description" yaml:"description"`
+		Version     string `json:"version" yaml:"version"`
+		AdminNoMutate bool `json:"admin-no-mutate" yaml:"admin-no-mutate"`
+	}{
+		Application: "Mutating-Webhook API",
+		Description: "Mutating Webhook for Simple Sidecar Injection",
+		Version:     "v1.0.0",
+	}
+	w.Header().Add(cT, cTjson)
+
 	for k, v := range urlPrams {
 		if k == "admin" && strings.ToLower(v[0]) == strings.ToLower(cfg.AllowAdminNoMutateToggle) {
-			if cfg.AllowAdminNoMutate {
-				cfg.AllowAdminNoMutate = false
-			} else {
-				cfg.AllowAdminNoMutate = true
-			}
+			cfg.AllowAdminNoMutate = !cfg.AllowAdminNoMutate
 			log.Printf("[INFO] Admin allow no mutate accepted current value: %v", cfg.AllowAdminNoMutate)
 		}
 	}
 
+	o.AdminNoMutate = cfg.AllowAdminNoMutate
+
 	output, err := json.MarshalIndent(o, "", "  ")
 	if err != nil {
 		log.Printf(marshalErrorMsg, err)