foomo/contentserver
1
0
Fork 0
mirror of https://github.com/foomo/contentserver.git synced 2025-10-16 12:25:44 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

fix: update persisted json after updating the repo using the poll routine

Browse Source
This commit is contained in:
Daniel Thomas 2025-06-13 22:21:59 +02:00
parent 74d2ff75f5
commit fe1d9de6f1
3 changed files with 11 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
pkg/handler/http.go
View File

@ -75,10 +75,6 @@ func (h *HTTP) ServeHTTP(w http.ResponseWriter, r *http.Request) {
route := Route(strings.TrimPrefix(r.URL.Path, h.basePath+"/")) route := Route(strings.TrimPrefix(r.URL.Path, h.basePath+"/"))
if route == RouteGetRepo { if route == RouteGetRepo {
resp := h.repo.Update()
if !resp.Success {
h.l.Warn("repo update failed — serving previously cached data", zap.String("reason", resp.ErrorMessage))
}
h.repo.WriteRepoBytes(w) h.repo.WriteRepoBytes(w)
w.Header().Set("Content-Type", "application/json") w.Header().Set("Content-Type", "application/json")
return return

9
pkg/repo/loader.go
View File

@ -305,6 +305,15 @@ func (r *Repo) update(ctx context.Context) (repoRuntime int64, err error) {
if r.poll { if r.poll {
r.pollVersion = repoURL r.pollVersion = repoURL
} }
// Persist the JSON buffer after successful update
if err := r.history.Add(r.JSONBufferBytes()); err != nil {
r.l.Error("Failed to persist repo after update", zap.Error(err))
metrics.HistoryPersistFailedCounter.WithLabelValues().Inc()
} else {
r.l.Info("Successfully persisted repo after update")
}
return repoRuntime, nil return repoRuntime, nil
} }

2
pkg/repo/repo.go
View File

@ -285,6 +285,8 @@ func (r *Repo) Update() (updateResponse *responses.Update) {
if historyErr != nil { if historyErr != nil {
r.l.Error("Could not persist current repo in history", zap.Error(historyErr)) r.l.Error("Could not persist current repo in history", zap.Error(historyErr))
metrics.HistoryPersistFailedCounter.WithLabelValues().Inc() metrics.HistoryPersistFailedCounter.WithLabelValues().Inc()
} else {
r.l.Info("Successfully persisted current repo to history")
} }
// add some stats // add some stats
for _, dimension := range r.Directory() { for _, dimension := range r.Directory() {