[cordova-android] branch master updated: fix(AllowListPlugin): Safely handle default allow navigation policy in allow request (#1342)

[er...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

erisu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/cordova-android.git


The following commit(s) were added to refs/heads/master by this push:
     new aea6b7f  fix(AllowListPlugin): Safely handle default allow navigation policy in allow request (#1342)
aea6b7f is described below

commit aea6b7f6f474efa256c77c318a4de8ca40baf7cd
Author: Joel Skrepnek <js...@gmail.com>
AuthorDate: Wed Sep 8 22:58:58 2021 -0700

    fix(AllowListPlugin): Safely handle default allow navigation policy in allow request (#1342)
---
 framework/src/org/apache/cordova/AllowListPlugin.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/framework/src/org/apache/cordova/AllowListPlugin.java b/framework/src/org/apache/cordova/AllowListPlugin.java
index 328a9b8..63dd5a1 100644
--- a/framework/src/org/apache/cordova/AllowListPlugin.java
+++ b/framework/src/org/apache/cordova/AllowListPlugin.java
@@ -122,7 +122,7 @@ public class AllowListPlugin extends CordovaPlugin {
 
     @Override
     public Boolean shouldAllowRequest(String url) {
-        return (this.shouldAllowNavigation(url) || this.allowedRequests.isUrlAllowListed(url))
+        return (Boolean.TRUE.equals(this.shouldAllowNavigation(url)) || this.allowedRequests.isUrlAllowListed(url))
             ? true
             : null; // default policy
     }

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cordova.apache.org
For additional commands, e-mail: commits-help@cordova.apache.org