Merge pull request #362 from JustAnotherArchivist/fix-metadata-rewrite

Only rewrite metadata file when the download counter changed
This commit is contained in:
Andrea Spacca
2021-04-24 21:23:51 +02:00
committed by GitHub

View File

@ -661,13 +661,11 @@ func (s *Server) CheckMetadata(token, filename string, increaseDownload bool) (M
return metadata, errors.New("MaxDownloads expired.") return metadata, errors.New("MaxDownloads expired.")
} else if !metadata.MaxDate.IsZero() && time.Now().After(metadata.MaxDate) { } else if !metadata.MaxDate.IsZero() && time.Now().After(metadata.MaxDate) {
return metadata, errors.New("MaxDate expired.") return metadata, errors.New("MaxDate expired.")
} else { } else if increaseDownload {
// todo(nl5887): mutex? // todo(nl5887): mutex?
// update number of downloads // update number of downloads
if increaseDownload { metadata.Downloads++
metadata.Downloads++
}
buffer := &bytes.Buffer{} buffer := &bytes.Buffer{}
if err := json.NewEncoder(buffer).Encode(metadata); err != nil { if err := json.NewEncoder(buffer).Encode(metadata); err != nil {