Avoid printing the whole JSON response if it is over 1024 characters in length (print the length instead). This was resulting in huge log files. In addition, log any failing unmarshalls of releases (at debug level) as well as the error (at warn level)
This commit is contained in:
1 parent
0543762330
commit
79efcb5e91
1 file changed
+21
-6
@@ -274,6 +274,12 @@ func safeUnpackReleases(
|
|||||||
"release_id", releaseID,
|
"release_id", releaseID,
|
||||||
"error", err,
|
"error", err,
|
||||||
)
|
)
|
||||||
|
logger.Debug(
|
||||||
|
"raw unreadable release",
|
||||||
|
"ocid", ocid,
|
||||||
|
"release_id", releaseID,
|
||||||
|
"release_raw", rawRelease,
|
||||||
|
)
|
||||||
continue
|
continue
|
||||||
}
|
}
|
||||||
out.Releases = append(out.Releases, release)
|
out.Releases = append(out.Releases, release)
|
||||||
@@ -347,12 +353,21 @@ func logTrace(logger *slog.Logger, res *resty.Response) {
|
|||||||
if logging.IsJSONHandler(logger.Handler()) {
|
if logging.IsJSONHandler(logger.Handler()) {
|
||||||
trace = res.Request.TraceInfo().JSON()
|
trace = res.Request.TraceInfo().JSON()
|
||||||
}
|
}
|
||||||
logger.Debug(
|
if len(res.String()) <= 1024 {
|
||||||
"fetched",
|
logger.Debug(
|
||||||
"trace", trace,
|
"fetched",
|
||||||
"status", res.Status(),
|
"trace", trace,
|
||||||
"response", res.String(),
|
"status", res.Status(),
|
||||||
)
|
"response", res.String(),
|
||||||
|
)
|
||||||
|
} else {
|
||||||
|
logger.Debug(
|
||||||
|
"fetched",
|
||||||
|
"trace", trace,
|
||||||
|
"status", res.Status(),
|
||||||
|
"response_length", len(res.String()),
|
||||||
|
)
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
// fetchURI performs a fetch against the API. If the API returns with a rate
|
// fetchURI performs a fetch against the API. If the API returns with a rate
|
||||||
|
|||||||
Reference in new issue
Block a user