outlook: fix temporary memory leak while listing events

As the whole request body is read using io.ReadAll, thus the request can be freed immediately afterwards.
inovex · Oct 10, 2024 · bd9f0b2 · bd9f0b2
1 parent ccdebc5
commit bd9f0b2
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/internal/adapter/outlook_http/client.go b/internal/adapter/outlook_http/client.go
@@ -46,7 +46,7 @@ func (o *OutlookClient) ListEvents(ctx context.Context, start time.Time, end tim
 	}
 
 	body, _ := io.ReadAll(resp.Body)
-	defer resp.Body.Close()
+	resp.Body.Close()
 
 	var eventList EventList
 	err = json.Unmarshal(body, &eventList)
@@ -62,7 +62,7 @@ func (o *OutlookClient) ListEvents(ctx context.Context, start time.Time, end tim
 		}
 
 		body, _ := io.ReadAll(resp.Body)
-		defer resp.Body.Close()
+		resp.Body.Close()
 
 		var nextList EventList
 		err = json.Unmarshal(body, &nextList)