jagan-parthiban commented on code in PR #7408: URL: https://github.com/apache/trafficcontrol/pull/7408#discussion_r1163684570
########## traffic_ops/traffic_ops_golang/servicecategory/servicecategories.go: ########## @@ -187,3 +191,317 @@ WHERE name=$2 RETURNING last_updated` func deleteQuery() string { return `DELETE FROM service_category WHERE name=:name` } + +// Get [Version : V5] function Process the *http.Request and writes the response. It uses GetServiceCategory function. +func Get(w http.ResponseWriter, r *http.Request) { + inf, userErr, sysErr, errCode := api.NewInfo(r, nil, nil) + if userErr != nil || sysErr != nil { + api.HandleErr(w, r, inf.Tx.Tx, errCode, userErr, sysErr) + return + } + defer inf.Close() + + code := http.StatusOK + useIMS := false + config, e := api.GetConfig(r.Context()) + if e == nil && config != nil { + useIMS = config.UseIMS + } else { + log.Warnf("Couldn't get config %v", e) + } + + var maxTime *time.Time + var err error + var scList []tc.ServiceCategoryV5 + + tx := inf.Tx + + scList, err, code, maxTime = GetServiceCategory(tx, inf.Params, useIMS, r.Header) + if code == http.StatusNotModified { + w.WriteHeader(code) + api.WriteResp(w, r, []tc.ServiceCategoryV5{}) + return + } + + if code == http.StatusBadRequest { + api.HandleErr(w, r, inf.Tx.Tx, http.StatusBadRequest, errors.New(err.Error()), nil) + return + } + + if err != nil { + api.HandleErr(w, r, inf.Tx.Tx, http.StatusInternalServerError, nil, errors.New("Service category errors: "+err.Error())) + return + } + + if maxTime != nil && api.SetLastModifiedHeader(r, useIMS) { + api.AddLastModifiedHdr(w, *maxTime) + } + + api.WriteResp(w, r, scList) +} + +// GetServiceCategory [Version : V5] receives transactions from Get function and returns service_categories list. +func GetServiceCategory(tx *sqlx.Tx, params map[string]string, useIMS bool, header http.Header) ([]tc.ServiceCategoryV5, error, int, *time.Time) { Review Comment: Moved the error to last value that is returned. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@trafficcontrol.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org