fix(aggregator): omit empty headers from the request

ncarlier · Jun 15, 2020 · e83c177 · e83c177
1 parent f4ec387
commit e83c177
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/pkg/aggregator/handler.go b/pkg/aggregator/handler.go
@@ -73,8 +73,12 @@ func (fh *FeedHandler) Refresh() (FeedStatus, []*model.Article) {
 	req.Header.Set("User-Agent", common.UserAgent)
 
 	// Add cache headers
-	req.Header.Add("If-Modified-Since", fh.status.LastModifiedHeader)
-	req.Header.Add("If-None-Match", fh.status.EtagHeader)
+	if fh.status.LastModifiedHeader != "" {
+		req.Header.Add("If-Modified-Since", fh.status.LastModifiedHeader)
+	}
+	if fh.status.EtagHeader != "" {
+		req.Header.Add("If-None-Match", fh.status.EtagHeader)
+	}
 
 	// Do HTTP call
 	resp, err := http.DefaultClient.Do(req)