Merge pull request #99 from proyecto26/98-prevent-ios-crash-when-redirect-url-is-null

Avoid stringifying null redirect url

Author: jdnichollsc

CHANGELOG.md

@@ -30,6 +30,7 @@ in case of vulnerabilities.
 ### Fixed
 - Complete migration to ns8 and fix build issues by [@rigor789](https://github.com/rigor789) ([#92](https://github.com/proyecto26/nativescript-inappbrowser/pull/92)).
 - Fix support for `Metadata Filtering` from Android by [@jcassidyav](https://github.com/jcassidyav) ([#93](https://github.com/proyecto26/nativescript-inappbrowser/pull/93)).
+- Avoid stringifying null redirect url by [@rmartin48](https://github.com/rmartin48) ([#99](https://github.com/proyecto26/nativescript-inappbrowser/pull/99)).
 
 ### Removed
 - Remove `QUERY_ALL_PACKAGES` permission by [@edusperoni](https://github.com/edusperoni) ([#87](https://github.com/proyecto26/nativescript-inappbrowser/pull/87)).

src/InAppBrowser.ios.ts

@@ -219,7 +219,10 @@ function setup() {
           if (!this.initializeWebBrowser(resolve, reject)) return;
 
           const url = NSURL.URLWithString(authUrl);
-          const escapedRedirectURL = NSURL.URLWithString(redirectUrl).scheme;
+          let escapedRedirectURL: string = null;
+          if (redirectUrl) {
+            escapedRedirectURL = NSURL.URLWithString(redirectUrl).scheme;
+          }
           this.authSession = (
             Utils.ios.MajorVersion >= 12
               ? ASWebAuthenticationSession