[webview_flutter_wkwebview] Fixes crash where the native AuthenticationChallengeResponse could not be found for auth requests

packages/webview_flutter/webview_flutter_wkwebview/CHANGELOG.md

@@ -1,3 +1,8 @@
+## 3.18.3
+
+* Fixes crash where the native `AuthenticationChallengeResponse` could not be found for auth
+  requests.
+
 ## 3.18.2
 
 * Updates generated pigeon code to ensure the internal wrapper immediately sends constructor calls.

packages/webview_flutter/webview_flutter_wkwebview/darwin/Tests/NavigationDelegateProxyAPITests.swift

@@ -219,17 +219,17 @@ class TestNavigationDelegateApi: PigeonApiProtocolWKNavigationDelegate {
     challenge challengeArg: URLAuthenticationChallenge,
     completion: @escaping (
       Result<
-        webview_flutter_wkwebview.AuthenticationChallengeResponse,
+        [Any?],
         webview_flutter_wkwebview.PigeonError
       >
     ) -> Void
   ) {
     didReceiveAuthenticationChallengeArgs = [webViewArg, challengeArg]
     completion(
-      .success(
-        AuthenticationChallengeResponse(
-          disposition: .useCredential,
-          credential: URLCredential(user: "user", password: "password", persistence: .none))))
+      .success([
+        UrlSessionAuthChallengeDisposition.useCredential,
+        URLCredential(user: "user", password: "password", persistence: .none),
+      ]))
   }
 }
 

packages/webview_flutter/webview_flutter_wkwebview/darwin/webview_flutter_wkwebview/Sources/webview_flutter_wkwebview/NavigationDelegateProxyAPIDelegate.swift

@@ -244,7 +244,23 @@ public class NavigationDelegateImpl: NSObject, WKNavigationDelegate {
           DispatchQueue.main.async {
             switch result {
             case .success(let response):
-              completionHandler(response.disposition, response.credential)
+              let disposition = response[0] as! UrlSessionAuthChallengeDisposition
+              var nativeDisposition: URLSession.AuthChallengeDisposition
+              switch disposition {
+              case .useCredential:
+                nativeDisposition = .useCredential
+              case .performDefaultHandling:
+                nativeDisposition = .performDefaultHandling
+              case .cancelAuthenticationChallenge:
+                nativeDisposition = .cancelAuthenticationChallenge
+              case .rejectProtectionSpace:
+                nativeDisposition = .rejectProtectionSpace
+              case .unknown:
+                print(
+                  self.registrar.createUnknownEnumError(withEnum: disposition).localizedDescription)
+                nativeDisposition = .cancelAuthenticationChallenge
+              }
+              completionHandler(nativeDisposition, response[1] as? URLCredential)
             case .failure(let error):
               completionHandler(.cancelAuthenticationChallenge, nil)
               onFailure("WKNavigationDelegate.didReceiveAuthenticationChallenge", error)
@@ -266,7 +282,23 @@ public class NavigationDelegateImpl: NSObject, WKNavigationDelegate {
           DispatchQueue.main.async {
             switch result {
             case .success(let response):
-              completionHandler(response.disposition, response.credential)
+              let disposition = response[0] as! UrlSessionAuthChallengeDisposition
+              var nativeDisposition: URLSession.AuthChallengeDisposition
+              switch disposition {
+              case .useCredential:
+                nativeDisposition = .useCredential
+              case .performDefaultHandling:
+                nativeDisposition = .performDefaultHandling
+              case .cancelAuthenticationChallenge:
+                nativeDisposition = .cancelAuthenticationChallenge
+              case .rejectProtectionSpace:
+                nativeDisposition = .rejectProtectionSpace
+              case .unknown:
+                print(
+                  self.registrar.createUnknownEnumError(withEnum: disposition).localizedDescription)
+                nativeDisposition = .cancelAuthenticationChallenge
+              }
+              completionHandler(nativeDisposition, response[1] as? URLCredential)
             case .failure(let error):
               completionHandler(.cancelAuthenticationChallenge, nil)
               onFailure("WKNavigationDelegate.didReceiveAuthenticationChallenge", error)

packages/webview_flutter/webview_flutter_wkwebview/darwin/webview_flutter_wkwebview/Sources/webview_flutter_wkwebview/WebKitLibrary.g.swift

@@ -3992,10 +3992,13 @@ protocol PigeonApiProtocolWKNavigationDelegate {
     pigeonInstance pigeonInstanceArg: WKNavigationDelegate, webView webViewArg: WKWebView,
     completion: @escaping (Result<Void, PigeonError>) -> Void)
   /// Asks the delegate to respond to an authentication challenge.
+  ///
+  /// Returns a List with a `UrlSessionAuthChallengeDisposition` and a nullable
+  /// `URLCredential`.
   func didReceiveAuthenticationChallenge(
     pigeonInstance pigeonInstanceArg: WKNavigationDelegate, webView webViewArg: WKWebView,
     challenge challengeArg: URLAuthenticationChallenge,
-    completion: @escaping (Result<AuthenticationChallengeResponse, PigeonError>) -> Void)
+    completion: @escaping (Result<[Any?], PigeonError>) -> Void)
 }
 
 final class PigeonApiWKNavigationDelegate: PigeonApiProtocolWKNavigationDelegate {
@@ -4333,10 +4336,13 @@ final class PigeonApiWKNavigationDelegate: PigeonApiProtocolWKNavigationDelegate
   }
 
   /// Asks the delegate to respond to an authentication challenge.
+  ///
+  /// Returns a List with a `UrlSessionAuthChallengeDisposition` and a nullable
+  /// `URLCredential`.
   func didReceiveAuthenticationChallenge(
     pigeonInstance pigeonInstanceArg: WKNavigationDelegate, webView webViewArg: WKWebView,
     challenge challengeArg: URLAuthenticationChallenge,
-    completion: @escaping (Result<AuthenticationChallengeResponse, PigeonError>) -> Void
+    completion: @escaping (Result<[Any?], PigeonError>) -> Void
   ) {
     if pigeonRegistrar.ignoreCallsToDart {
       completion(
@@ -4369,7 +4375,7 @@ final class PigeonApiWKNavigationDelegate: PigeonApiProtocolWKNavigationDelegate
               code: "null-error",
               message: "Flutter api returned null value for non-null return value.", details: "")))
       } else {
-        let result = listResponse[0] as! AuthenticationChallengeResponse
+        let result = listResponse[0] as! [Any?]
         completion(.success(result))
       }
     }

packages/webview_flutter/webview_flutter_wkwebview/lib/src/common/web_kit.g.dart

@@ -4326,6 +4326,9 @@ class WKNavigationDelegate extends NSObject {
 
   /// Asks the delegate to respond to an authentication challenge.
   ///
+  /// Returns a List with a `UrlSessionAuthChallengeDisposition` and a nullable
+  /// `URLCredential`.
+  ///
   /// For the associated Native object to be automatically garbage collected,
   /// it is required that the implementation of this `Function` doesn't have a
   /// strong reference to the encapsulating class instance. When this `Function`
@@ -4343,7 +4346,7 @@ class WKNavigationDelegate extends NSObject {
   ///
   /// Alternatively, [PigeonInstanceManager.removeWeakReference] can be used to
   /// release the associated Native object manually.
-  final Future<AuthenticationChallengeResponse> Function(
+  final Future<List<Object?>> Function(
     WKNavigationDelegate pigeon_instance,
     WKWebView webView,
     URLAuthenticationChallenge challenge,
@@ -4387,7 +4390,7 @@ class WKNavigationDelegate extends NSObject {
       WKNavigationDelegate pigeon_instance,
       WKWebView webView,
     )? webViewWebContentProcessDidTerminate,
-    Future<AuthenticationChallengeResponse> Function(
+    Future<List<Object?>> Function(
       WKNavigationDelegate pigeon_instance,
       WKWebView webView,
       URLAuthenticationChallenge challenge,
@@ -4693,7 +4696,7 @@ class WKNavigationDelegate extends NSObject {
           assert(arg_challenge != null,
               'Argument for dev.flutter.pigeon.webview_flutter_wkwebview.WKNavigationDelegate.didReceiveAuthenticationChallenge was null, expected non-null URLAuthenticationChallenge.');
           try {
-            final AuthenticationChallengeResponse output =
+            final List<Object?> output =
                 await (didReceiveAuthenticationChallenge ??
                         arg_pigeon_instance!.didReceiveAuthenticationChallenge)
                     .call(arg_pigeon_instance!, arg_webView!, arg_challenge!);

packages/webview_flutter/webview_flutter_wkwebview/lib/src/legacy/web_kit_webview_widget.dart

@@ -193,10 +193,10 @@ class WebKitWebViewPlatformController extends WebViewPlatformController {
           return NavigationResponsePolicy.allow;
         },
         didReceiveAuthenticationChallenge: (_, __, ___) async {
-          return AuthenticationChallengeResponse(
-            disposition:
-                UrlSessionAuthChallengeDisposition.performDefaultHandling,
-          );
+          return <Object?>[
+            UrlSessionAuthChallengeDisposition.performDefaultHandling,
+            null,
+          ];
         },
       );
     },
@@ -751,7 +751,7 @@ class WebViewWidgetProxy {
       WKWebView webView,
       WKNavigationResponse navigationResponse,
     ) decidePolicyForNavigationResponse,
-    required Future<AuthenticationChallengeResponse> Function(
+    required Future<List<Object?>> Function(
       WKNavigationDelegate,
       WKWebView webView,
       URLAuthenticationChallenge challenge,

packages/webview_flutter/webview_flutter_wkwebview/lib/src/webkit_proxy.dart

@@ -102,7 +102,7 @@ class WebKitProxy {
       WKNavigationDelegate,
       WKWebView,
     )? webViewWebContentProcessDidTerminate,
-    required Future<AuthenticationChallengeResponse> Function(
+    required Future<List<Object?>> Function(
       WKNavigationDelegate,
       WKWebView,
       URLAuthenticationChallenge,

packages/webview_flutter/webview_flutter_wkwebview/lib/src/webkit_webview_controller.dart

@@ -1213,43 +1213,46 @@ class WebKitNavigationDelegate extends PlatformNavigationDelegate {
           final String host = protectionSpace.host;
           final String? realm = protectionSpace.realm;
 
-          final Completer<AuthenticationChallengeResponse> responseCompleter =
-              Completer<AuthenticationChallengeResponse>();
+          final Completer<List<Object?>> responseCompleter =
+              Completer<List<Object?>>();
 
           callback(
             HttpAuthRequest(
               host: host,
               realm: realm,
               onProceed: (WebViewCredential credential) {
-                final AuthenticationChallengeResponse response =
-                    proxy.newAuthenticationChallengeResponse(
-                  disposition: UrlSessionAuthChallengeDisposition.useCredential,
-                  credential: URLCredential.withUser(
-                    user: credential.user,
-                    password: credential.password,
-                    persistence: UrlCredentialPersistence.forSession,
-                  ),
+                final URLCredential userCredential =
+                    proxy.withUserURLCredential(
+                  user: credential.user,
+                  password: credential.password,
+                  persistence: UrlCredentialPersistence.forSession,
+                );
+                responseCompleter.complete(
+                  <Object?>[
+                    UrlSessionAuthChallengeDisposition.useCredential,
+                    userCredential,
+                  ],
                 );
-                responseCompleter.complete(response);
               },
               onCancel: () {
-                final AuthenticationChallengeResponse response =
-                    proxy.newAuthenticationChallengeResponse(
-                  disposition: UrlSessionAuthChallengeDisposition
-                      .cancelAuthenticationChallenge,
+                responseCompleter.complete(
+                  <Object?>[
+                    UrlSessionAuthChallengeDisposition
+                        .cancelAuthenticationChallenge,
+                    null,
+                  ],
                 );
-                responseCompleter.complete(response);
               },
             ),
           );
 
           return responseCompleter.future;
         }
 
-        return AuthenticationChallengeResponse(
-          disposition:
-              UrlSessionAuthChallengeDisposition.performDefaultHandling,
-        );
+        return <Object?>[
+          UrlSessionAuthChallengeDisposition.performDefaultHandling,
+          null,
+        ];
       },
     );
   }

packages/webview_flutter/webview_flutter_wkwebview/pigeons/web_kit.dart

@@ -745,9 +745,15 @@ abstract class WKNavigationDelegate extends NSObject {
   /// Tells the delegate that the web view’s content process was terminated.
   void Function(WKWebView webView)? webViewWebContentProcessDidTerminate;
 
+  // TODO(bparrishMines): This method should return an
+  // `AuthenticationChallengeResponse` once the cause of
+  // https://github.com/flutter/flutter/issues/162437 can be found and fixed.
   /// Asks the delegate to respond to an authentication challenge.
+  ///
+  /// This return value expects a List with a
+  /// `UrlSessionAuthChallengeDisposition` and a nullable `URLCredential`.
   @async
-  late AuthenticationChallengeResponse Function(
+  late List<Object?> Function(
     WKWebView webView,
     URLAuthenticationChallenge challenge,
   ) didReceiveAuthenticationChallenge;

packages/webview_flutter/webview_flutter_wkwebview/pubspec.yaml

@@ -2,7 +2,7 @@ name: webview_flutter_wkwebview
 description: A Flutter plugin that provides a WebView widget based on Apple's WKWebView control.
 repository: https://github.com/flutter/packages/tree/main/packages/webview_flutter/webview_flutter_wkwebview
 issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+webview%22
-version: 3.18.2
+version: 3.18.3
 
 environment:
   sdk: ^3.5.0