Skip to content

Commit

Permalink
fix(AllowListPlugin): Safely handle default allow navigation policy i…
Browse files Browse the repository at this point in the history
…n allow request (apache#1342)
  • Loading branch information
jskrepnek authored and wedgberto committed May 17, 2022
1 parent 491af91 commit 46d2c69
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion framework/src/org/apache/cordova/AllowListPlugin.java
Original file line number Diff line number Diff line change
Expand Up @@ -122,7 +122,7 @@ public Boolean shouldAllowNavigation(String url) {

@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
}
Expand Down

0 comments on commit 46d2c69

Please sign in to comment.