Change so that we only send 300 markers per request. Fixes #3711

Account/Sources/Account/Feedly/FeedlyAPICaller.swift

@@ -7,6 +7,7 @@
 //
 
 import Foundation
+import RSCore
 import RSWeb
 import Secrets
 
@@ -820,33 +821,48 @@ extension FeedlyAPICaller: FeedlyMarkArticlesService {
 			fatalError("\(components) does not produce a valid URL.")
 		}
 		
-		var request = URLRequest(url: url)
+		let articleIdChunks = Array(articleIds).chunked(into: 300)
-		request.httpMethod = "POST"
+		let dispatchGroup = DispatchGroup()
-		request.addValue("application/json", forHTTPHeaderField: HTTPRequestHeader.contentType)
+		var groupError: Error? = nil
-		request.addValue("application/json", forHTTPHeaderField: "Accept-Type")
+
-		request.addValue("OAuth \(accessToken)", forHTTPHeaderField: HTTPRequestHeader.authorization)
+		for articleIdChunk in articleIdChunks {
-		
+			
-		do {
+			var request = URLRequest(url: url)
-			let body = MarkerEntriesBody(action: action.actionValue, entryIds: Array(articleIds))
+			request.httpMethod = "POST"
-			let encoder = JSONEncoder()
+			request.addValue("application/json", forHTTPHeaderField: HTTPRequestHeader.contentType)
-			let data = try encoder.encode(body)
+			request.addValue("application/json", forHTTPHeaderField: "Accept-Type")
-			request.httpBody = data
+			request.addValue("OAuth \(accessToken)", forHTTPHeaderField: HTTPRequestHeader.authorization)
-		} catch {
+			
-			return DispatchQueue.main.async {
+			do {
-				completion(.failure(error))
+				let body = MarkerEntriesBody(action: action.actionValue, entryIds: Array(articleIdChunk))
+				let encoder = JSONEncoder()
+				let data = try encoder.encode(body)
+				request.httpBody = data
+			} catch {
+				return DispatchQueue.main.async {
+					completion(.failure(error))
+				}
+			}
+			
+			dispatchGroup.enter()
+			send(request: request, resultType: String.self, dateDecoding: .millisecondsSince1970, keyDecoding: .convertFromSnakeCase) { result in
+				switch result {
+				case .success(let (httpResponse, _)):
+					if httpResponse.statusCode != 200 {
+						groupError = URLError(.cannotDecodeContentData)
+					}
+				case .failure(let error):
+					groupError = error
+				}
+				dispatchGroup.leave()
 			}
 		}
 		
-		send(request: request, resultType: String.self, dateDecoding: .millisecondsSince1970, keyDecoding: .convertFromSnakeCase) { result in
+		dispatchGroup.notify(queue: .main) {
-			switch result {
+			if let groupError = groupError {
-			case .success(let (httpResponse, _)):
+				completion(.failure(groupError))
-				if httpResponse.statusCode == 200 {
+			} else {
-					completion(.success(()))
+				completion(.success(()))
-				} else {
-					completion(.failure(URLError(.cannotDecodeContentData)))
-				}
-			case .failure(let error):
-				completion(.failure(error))
 			}
 		}
 	}