diff --git a/adminapi/telemetry/telemetry_profile_controller.go b/adminapi/telemetry/telemetry_profile_controller.go index 205476a..1efefdf 100644 --- a/adminapi/telemetry/telemetry_profile_controller.go +++ b/adminapi/telemetry/telemetry_profile_controller.go @@ -77,6 +77,7 @@ func CreateTelemetryEntryFor(w http.ResponseWriter, r *http.Request) { } xw, ok := w.(*xwhttp.XResponseWriter) if !ok { + log.Error("Unable to cast responsewriter to XResponseWriter in CreateTelemetryEntryFor") xwhttp.Error(w, http.StatusInternalServerError, xwcommon.NewRemoteErrorAS(http.StatusInternalServerError, "responsewriter cast error")) return } diff --git a/adminapi/telemetry/telemetry_rule_handler.go b/adminapi/telemetry/telemetry_rule_handler.go index 723f43b..8f484be 100644 --- a/adminapi/telemetry/telemetry_rule_handler.go +++ b/adminapi/telemetry/telemetry_rule_handler.go @@ -23,6 +23,7 @@ import ( "net/http" "github.com/gorilla/mux" + log "github.com/sirupsen/logrus" xutil "github.com/rdkcentral/xconfadmin/util" @@ -183,6 +184,7 @@ func UpdateTelemetryRuleHandler(w http.ResponseWriter, r *http.Request) { // r.Body is already drained in the middleware xw, ok := w.(*xwhttp.XResponseWriter) if !ok { + log.Error("failed to cast responsewriter to XResponseWriter in UpdateTelemetryRuleHandler") xhttp.WriteAdminErrorResponse(w, http.StatusInternalServerError, "unable to cast XResponseWriter object") return } diff --git a/adminapi/telemetry/telemetry_rule_service.go b/adminapi/telemetry/telemetry_rule_service.go index c14ad5e..d5bea1f 100644 --- a/adminapi/telemetry/telemetry_rule_service.go +++ b/adminapi/telemetry/telemetry_rule_service.go @@ -26,6 +26,7 @@ import ( "strings" ru "github.com/rdkcentral/xconfwebconfig/rulesengine" + log "github.com/sirupsen/logrus" queries "github.com/rdkcentral/xconfadmin/adminapi/queries" xcommon "github.com/rdkcentral/xconfadmin/common" @@ -66,6 +67,7 @@ func DeleteTelemetryRulebyId(id string, app string) *xwhttp.ResponseEntity { err = DeleteOneTelemetryRule(id) if err != nil { + log.WithFields(log.Fields{"entity_id": id, "error": err}).Error("Failed to delete telemetry rule") return xwhttp.NewResponseEntity(http.StatusInternalServerError, err, nil) } @@ -159,6 +161,7 @@ func CreateTelemetryRule(tmrule *xwlogupload.TelemetryRule, app string) *xwhttp. tmrule.Updated = xwutil.GetTimestamp() if err := db.GetCachedSimpleDao().SetOne(db.TABLE_TELEMETRY_RULES, tmrule.ID, tmrule); err != nil { + log.WithFields(log.Fields{"entity_id": tmrule.ID, "error": err}).Error("Failed to save telemetry rule") return xwhttp.NewResponseEntity(http.StatusInternalServerError, err, nil) } @@ -187,6 +190,7 @@ func UpdateTelemetryRule(tmrule *xwlogupload.TelemetryRule, app string) *xwhttp. tmrule.Updated = xwutil.GetTimestamp() if err = db.GetCachedSimpleDao().SetOne(db.TABLE_TELEMETRY_RULES, tmrule.ID, tmrule); err != nil { + log.WithFields(log.Fields{"entity_id": tmrule.ID, "error": err}).Error("Failed to update telemetry rule") return xwhttp.NewResponseEntity(http.StatusInternalServerError, err, nil) } diff --git a/adminapi/telemetry/telemetry_v2_rule_handler.go b/adminapi/telemetry/telemetry_v2_rule_handler.go index 7e4825a..f7a7ca0 100644 --- a/adminapi/telemetry/telemetry_v2_rule_handler.go +++ b/adminapi/telemetry/telemetry_v2_rule_handler.go @@ -170,6 +170,7 @@ func GetTelemetryTwoRulesFilteredWithPage(w http.ResponseWriter, r *http.Request } xw, ok := w.(*xwhttp.XResponseWriter) if !ok { + log.Error("Failed to cast response writer to XResponseWriter in GetTelemetryTwoRulesFilteredWithPage") xhttp.AdminError(w, xwcommon.NewRemoteErrorAS(http.StatusInternalServerError, "responsewriter cast error")) return } @@ -202,6 +203,7 @@ func CreateTelemetryTwoRuleHandler(w http.ResponseWriter, r *http.Request) { // r.Body is already drained in the middleware xw, ok := w.(*xwhttp.XResponseWriter) if !ok { + log.Error("Unable to cast responsewriter to XResponseWriter in CreateTelemetryTwoRuleHandler") xhttp.AdminError(w, xwcommon.NewRemoteErrorAS(http.StatusInternalServerError, "responsewriter cast error")) return } @@ -282,6 +284,7 @@ func UpdateTelemetryTwoRuleHandler(w http.ResponseWriter, r *http.Request) { // r.Body is already drained in the middleware xw, ok := w.(*xwhttp.XResponseWriter) if !ok { + log.Error("Unable to cast responsewriter to XResponseWriter in UpdateTelemetryTwoRuleHandler") xhttp.AdminError(w, xwcommon.NewRemoteErrorAS(http.StatusInternalServerError, "responsewriter cast error")) return }