fix: general quality of service improvements

[CodingAleCR]

No description provided.

[cursor]

Bug: CopyWith Mutates Original MultipartRequest Files

The MultipartRequest.copyWith method calls file.finalize() on the original MultipartFile objects. Since finalize() consumes the file's stream and can only be called once, this makes the original request's files unusable and can result in the copied request having invalid or empty file contents if the originals were already finalized. This violates the non-mutating behavior expected from copyWith().

lib/extensions/multipart_request.dart#L25-L44

http_interceptor/lib/extensions/multipart_request.dart

Lines 25 to 44 in 7a57b40

	for (var file in this.files) {
	clonedRequest.files.add(MultipartFile(
	file.field,
	file.finalize(),
	file.length,
	filename: file.filename,
	contentType: file.contentType,
	));
	}
	} else {
	// Use the provided files
	for (var file in files) {
	clonedRequest.files.add(MultipartFile(
	file.field,
	file.finalize(),
	file.length,
	filename: file.filename,
	contentType: file.contentType,
	));
	}

Fix in Cursor • Fix in Web

---

Was this report helpful? Give feedback by reacting with 👍 or 👎

[CodingAleCR]

‼ This one might get superseeded by #173 since I'd rather improve the rebuilt version than keep fixing issues on 2.x versions. The factor of time that I can spend working on this is also a factor for that.