mirror of
https://github.com/project-zot/zot.git
synced 2024-12-16 21:56:37 -05:00
routes: add more logs
There still appears to be another path where we are seeing 500s. Adding more logs to catch this path.
This commit is contained in:
parent
2b9d29c9a2
commit
31b30d858b
2 changed files with 6 additions and 4 deletions
|
@ -267,7 +267,9 @@ func (rh *RouteHandler) UpdateManifest(w http.ResponseWriter, r *http.Request) {
|
|||
|
||||
body, err := ioutil.ReadAll(r.Body)
|
||||
if err != nil {
|
||||
rh.c.Log.Error().Err(err).Msg("unexpected error")
|
||||
w.WriteHeader(http.StatusInternalServerError)
|
||||
|
||||
return
|
||||
}
|
||||
|
||||
|
@ -605,7 +607,6 @@ func (rh *RouteHandler) GetBlobUpload(w http.ResponseWriter, r *http.Request) {
|
|||
// @Failure 500 {string} string "internal server error"
|
||||
// @Router /v2/{name}/blobs/uploads/{uuid} [patch]
|
||||
func (rh *RouteHandler) PatchBlobUpload(w http.ResponseWriter, r *http.Request) {
|
||||
rh.c.Log.Info().Interface("headers", r.Header).Msg("request headers")
|
||||
vars := mux.Vars(r)
|
||||
name, ok := vars["name"]
|
||||
|
||||
|
@ -894,7 +895,7 @@ func WriteJSON(w http.ResponseWriter, status int, data interface{}) {
|
|||
body, err := json.Marshal(data)
|
||||
|
||||
if err != nil {
|
||||
w.WriteHeader(http.StatusInternalServerError)
|
||||
panic(err)
|
||||
}
|
||||
|
||||
WriteData(w, status, DefaultMediaType, body)
|
||||
|
@ -920,7 +921,8 @@ func WriteDataFromReader(w http.ResponseWriter, status int, length int64, mediaT
|
|||
break
|
||||
} else if err != nil {
|
||||
// other kinds of intermittent errors can occur, e.g, io.ErrShortWrite
|
||||
logger.Panic().Err(err).Msg("copying data into http response")
|
||||
logger.Error().Err(err).Msg("copying data into http response")
|
||||
return
|
||||
}
|
||||
}
|
||||
}
|
||||
|
|
|
@ -40,7 +40,7 @@ func NewLogger(level string, output string) Logger {
|
|||
log = zerolog.New(file)
|
||||
}
|
||||
|
||||
return Logger{Logger: log.With().Timestamp().Logger()}
|
||||
return Logger{Logger: log.With().Caller().Timestamp().Logger()}
|
||||
}
|
||||
|
||||
type statusWriter struct {
|
||||
|
|
Loading…
Reference in a new issue