fix: intercepting body causes headers to be reset

Author: CodingAleCR

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 2.0.0-beta.2
+
+- 🐞  Fixed: Changing `body` causes that the `headers` also change and ignore previous interceptions (i.e. content-type headers are overriden).
+
 ## 2.0.0-beta.1
 
 - ❗️🛠  Changed: Renamed `Method` to use `HttpMethod` and refactored helper functions into extensions (`StringToMethod`, and `MethodToString`).

lib/extensions/request.dart

@@ -21,9 +21,9 @@ extension RequestCopyWith on Request {
         method?.asString ?? this.method,
         url ?? this.url,
       )
-        ..headers.addAll(headers ?? {})
-        ..encoding = encoding ?? this.encoding
         ..body = body ?? this.body
+        ..headers.addAll(headers ?? this.headers)
+        ..encoding = encoding ?? this.encoding
         ..followRedirects = followRedirects ?? this.followRedirects
         ..maxRedirects = maxRedirects ?? this.maxRedirects
         ..persistentConnection =

pubspec.yaml

@@ -1,6 +1,6 @@
 name: http_interceptor
 description: A lightweight, simple plugin that allows you to intercept request and response objects and modify them if desired.
-version: 2.0.0-beta.1
+version: 2.0.0-beta.2
 homepage: https://github.com/CodingAleCR/http_interceptor
 issue_tracker: https://github.com/CodingAleCR/http_interceptor/issues
 repository: https://github.com/CodingAleCR/http_interceptor

test/extensions/request_test.dart

@@ -1,4 +1,5 @@
 import 'dart:convert';
+import 'dart:io';
 
 import 'package:http/http.dart';
 import 'package:http_interceptor/http_interceptor.dart';
@@ -10,7 +11,10 @@ main() {
 
   setUpAll(() {
     baseRequest = Request("GET", Uri.https("www.google.com", "/helloworld"))
-      ..body = jsonEncode(<String, String>{'some_param': 'some value'});
+      ..body = jsonEncode(<String, String>{'some_param': 'some value'})
+      ..headers.addAll({
+        HttpHeaders.contentTypeHeader: 'application/json; charset=utf-8',
+      });
     request = baseRequest as Request;
   });
 
@@ -154,7 +158,7 @@ main() {
     test('Request is copied overriding headers', () {
       // Arrange
       final newHeaders = Map<String, String>.from(request.headers);
-      newHeaders['content-type'] = 'application/json; charset=utf-8';
+      newHeaders['content-type'] = 'text/plain; charset=utf-8';
 
       // Act
       Request copied = request.copyWith(
@@ -202,10 +206,13 @@ main() {
       expect(copied.maxRedirects, equals(request.maxRedirects));
       expect(copied.persistentConnection, equals(request.persistentConnection));
     });
+
     test('Request is copied with different encoding', () {
       // Arrange
       final newEncoding = Encoding.getByName('latin1');
-      final changedHeaders = {'content-type': 'text/plain; charset=iso-8859-1'};
+      final changedHeaders = {
+        'content-type': 'application/json; charset=iso-8859-1'
+      };
 
       // Act
       Request copied = request.copyWith(