Added the ability to retrieve url from BaseResponse

lib/src/base_request.dart

@@ -136,6 +136,7 @@ abstract class BaseRequest {
           contentLength: response.contentLength,
           request: response.request,
           headers: response.headers,
+          url: response.url,
           isRedirect: response.isRedirect,
           persistentConnection: response.persistentConnection,
           reasonPhrase: response.reasonPhrase);

lib/src/base_response.dart

@@ -30,6 +30,7 @@ abstract class BaseResponse {
   final Map<String, String> headers;
 
   final bool isRedirect;
+  final String? url;
 
   /// Whether the server requested that a persistent connection be maintained.
   final bool persistentConnection;
@@ -39,6 +40,7 @@ abstract class BaseResponse {
       this.request,
       this.headers = const {},
       this.isRedirect = false,
+      this.url,
       this.persistentConnection = true,
       this.reasonPhrase}) {
     if (statusCode < 100) {

lib/src/browser_client.dart

@@ -59,6 +59,7 @@ class BrowserClient extends BaseClient {
           contentLength: body.length,
           request: request,
           headers: xhr.responseHeaders,
+          url: xhr.responseUrl,
           reasonPhrase: xhr.statusText));
     }));
 

lib/src/io_client.dart

@@ -58,6 +58,9 @@ class IOClient extends BaseClient {
               response.contentLength == -1 ? null : response.contentLength,
           request: request,
           headers: headers,
+          url: response.redirects.isEmpty
+              ? request.url.toString()
+              : response.redirects.last.location.toString(),
           isRedirect: response.isRedirect,
           persistentConnection: response.persistentConnection,
           reasonPhrase: response.reasonPhrase,

lib/src/io_streamed_response.dart

@@ -24,12 +24,14 @@ class IOStreamedResponse extends StreamedResponse {
       bool isRedirect = false,
       bool persistentConnection = true,
       String? reasonPhrase,
+      String? url,
       HttpClientResponse? inner})
       : _inner = inner,
         super(stream, statusCode,
             contentLength: contentLength,
             request: request,
             headers: headers,
+            url: url,
             isRedirect: isRedirect,
             persistentConnection: persistentConnection,
             reasonPhrase: reasonPhrase);

lib/src/response.dart

@@ -32,11 +32,13 @@ class Response extends BaseResponse {
       {BaseRequest? request,
       Map<String, String> headers = const {},
       bool isRedirect = false,
+      String? url,
       bool persistentConnection = true,
       String? reasonPhrase})
       : this.bytes(_encodingForHeaders(headers).encode(body), statusCode,
             request: request,
             headers: headers,
+            url: url,
             isRedirect: isRedirect,
             persistentConnection: persistentConnection,
             reasonPhrase: reasonPhrase);
@@ -46,13 +48,15 @@ class Response extends BaseResponse {
       {BaseRequest? request,
       Map<String, String> headers = const {},
       bool isRedirect = false,
+      String? url,
       bool persistentConnection = true,
       String? reasonPhrase})
       : bodyBytes = toUint8List(bodyBytes),
         super(statusCode,
             contentLength: bodyBytes.length,
             request: request,
             headers: headers,
+            url: url,
             isRedirect: isRedirect,
             persistentConnection: persistentConnection,
             reasonPhrase: reasonPhrase);
@@ -64,6 +68,7 @@ class Response extends BaseResponse {
     return Response.bytes(body, response.statusCode,
         request: response.request,
         headers: response.headers,
+        url: response.url,
         isRedirect: response.isRedirect,
         persistentConnection: response.persistentConnection,
         reasonPhrase: response.reasonPhrase);

lib/src/streamed_response.dart

@@ -23,13 +23,15 @@ class StreamedResponse extends BaseResponse {
       BaseRequest? request,
       Map<String, String> headers = const {},
       bool isRedirect = false,
+      String? url,
       bool persistentConnection = true,
       String? reasonPhrase})
       : stream = toByteStream(stream),
         super(statusCode,
             contentLength: contentLength,
             request: request,
             headers: headers,
+            url: url,
             isRedirect: isRedirect,
             persistentConnection: persistentConnection,
             reasonPhrase: reasonPhrase);

pubspec.yaml

@@ -1,5 +1,5 @@
 name: http
-version: 0.13.4-dev
+version: 0.13.4
 homepage: https://github.com/dart-lang/http
 description: A composable, multi-platform, Future-based API for HTTP requests.
 

test/io/request_test.dart

@@ -22,6 +22,7 @@ void main() {
     final response = await request.send();
 
     expect(response.statusCode, equals(200));
+    expect(response.url, equals(serverUrl.toString()));
     final bytesString = await response.stream.bytesToString();
     expect(
         bytesString,
@@ -44,13 +45,15 @@ void main() {
     final response = await request.send();
 
     expect(response.statusCode, equals(302));
+    expect(response.url, equals(serverUrl.resolve('/redirect').toString()));
   });
 
   test('with redirects', () async {
     final request = http.Request('GET', serverUrl.resolve('/redirect'));
     final response = await request.send();
 
     expect(response.statusCode, equals(200));
+    expect(response.url, equals(serverUrl.resolve('/').toString()));
     final bytesString = await response.stream.bytesToString();
     expect(bytesString, parse(containsPair('path', '/')));
   });