improved serving of small and large (tested ~1Mb) cover images

src/Web.cpp

@@ -1937,18 +1937,14 @@ static void handleCoverImageRequest(AsyncWebServerRequest *request) {
 	}
 
 	int imageSize = gPlayProperties.coverFileSize;
-	AsyncWebServerResponse *response = request->beginChunkedResponse(mimeType, [coverFile, imageSize](uint8_t *buffer, size_t maxLen, size_t index) -> size_t {
-		// some kind of webserver bug with actual size available, reduce the len
-		maxLen = maxLen >> 1;
-
+	AsyncWebServerResponse *response = request->beginResponse(mimeType, imageSize, [coverFile, imageSize](uint8_t *buffer, size_t maxLen, size_t index) -> size_t {
 		File file = coverFile; // local copy of file pointer
-		size_t leftToWrite = imageSize - index;
-		if (!leftToWrite) {
-			return 0; // end of transfer
-		}
-		size_t willWrite = (leftToWrite > maxLen) ? maxLen : leftToWrite;
+
+		// read a maximum of 2048 bytes to avoid blocking too long
+		size_t willWrite = min((size_t) 2048, maxLen);
+
 		file.read(buffer, willWrite);
-		index += willWrite;
+
 		return willWrite;
 	});
 	response->addHeader("Cache Control", "no-cache, must-revalidate");