fix: Stop sending before logs when after is specified (#2585)

This fixes duplicate logs appearing in completed jos!
This commit is contained in:
Kyle Carberry
2022-06-22 12:09:28 -05:00
committed by GitHub
parent 437066ce20
commit caf9c41a9e

View File

@ -72,8 +72,11 @@ func (api *API) provisionerJobLogs(rw http.ResponseWriter, r *http.Request, job
} }
before = time.UnixMilli(beforeMS) before = time.UnixMilli(beforeMS)
} else { } else {
// If we're following, we don't want logs before a timestamp!
if !follow {
before = database.Now() before = database.Now()
} }
}
if !follow { if !follow {
logs, err := api.Database.GetProvisionerLogsByIDBetween(r.Context(), database.GetProvisionerLogsByIDBetweenParams{ logs, err := api.Database.GetProvisionerLogsByIDBetween(r.Context(), database.GetProvisionerLogsByIDBetweenParams{