resolved comments

Author: msafwankarim

internal/pkg/handler/update.go

@@ -41,17 +41,18 @@ func (r ResourceUpdatedHandler) Handle() error {
 func (r ResourceUpdatedHandler) GetConfig() (common.Config, string) {
 	var oldSHAData string
 	var config common.Config
-	if _, ok := r.Resource.(*v1.ConfigMap); ok {
+	switch res := r.Resource.(type) {
+	case *v1.ConfigMap:
 		oldSHAData = util.GetSHAfromConfigmap(r.OldResource.(*v1.ConfigMap))
-		config = common.GetConfigmapConfig(r.Resource.(*v1.ConfigMap))
-	} else if _, ok := r.Resource.(*v1.Secret); ok {
+		config = common.GetConfigmapConfig(res)
+	case *v1.Secret:
 		oldSHAData = util.GetSHAfromSecret(r.OldResource.(*v1.Secret).Data)
-		config = common.GetSecretConfig(r.Resource.(*v1.Secret))
-	} else if _, ok := r.Resource.(*csiv1.SecretProviderClassPodStatus); ok {
+		config = common.GetSecretConfig(res)
+	case *csiv1.SecretProviderClassPodStatus:
 		oldSHAData = util.GetSHAfromSecretProviderClassPodStatus(r.OldResource.(*csiv1.SecretProviderClassPodStatus).Status)
-		config = common.GetSecretProviderClassPodStatusConfig(r.Resource.(*csiv1.SecretProviderClassPodStatus))
-	} else {
-		logrus.Warnf("Invalid resource: Resource should be 'Secret' or 'Configmap' but found, %v", r.Resource)
+		config = common.GetSecretProviderClassPodStatusConfig(res)
+	default:
+		logrus.Warnf("Invalid resource: Resource should be 'Secret', 'Configmap' or 'SecretProviderClassPodStatus' but found, %v", r.Resource)
 	}
 	return config, oldSHAData
 }