Fix panic in conversion webhook

Author: Tomy2e

pkg/webhook/conversion/conversion.go

@@ -31,6 +31,7 @@ import (
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 	"k8s.io/apimachinery/pkg/runtime"
 	"k8s.io/apimachinery/pkg/runtime/schema"
+	"k8s.io/apimachinery/pkg/types"
 	"sigs.k8s.io/controller-runtime/pkg/conversion"
 	logf "sigs.k8s.io/controller-runtime/pkg/log"
 )
@@ -69,16 +70,21 @@ func (wh *Webhook) ServeHTTP(w http.ResponseWriter, r *http.Request) {
 		return
 	}
 
+	var uid types.UID
+	if convertReview.Request != nil {
+		uid = convertReview.Request.UID
+	}
+
 	// TODO(droot): may be move the conversion logic to a separate module to
 	// decouple it from the http layer ?
 	resp, err := wh.handleConvertRequest(convertReview.Request)
 	if err != nil {
-		log.Error(err, "failed to convert", "request", convertReview.Request.UID)
+		log.Error(err, "failed to convert", "request", uid)
 		convertReview.Response = errored(err)
 	} else {
 		convertReview.Response = resp
 	}
-	convertReview.Response.UID = convertReview.Request.UID
+	convertReview.Response.UID = uid
 	convertReview.Request = nil
 
 	err = json.NewEncoder(w).Encode(convertReview)