You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cordova.apache.org by ia...@apache.org on 2014/10/30 17:20:37 UTC
[2/7] android commit: Remove whitelist config.xml parsing
Remove whitelist config.xml parsing
Project: http://git-wip-us.apache.org/repos/asf/cordova-android/repo
Commit: http://git-wip-us.apache.org/repos/asf/cordova-android/commit/ed78b557
Tree: http://git-wip-us.apache.org/repos/asf/cordova-android/tree/ed78b557
Diff: http://git-wip-us.apache.org/repos/asf/cordova-android/diff/ed78b557
Branch: refs/heads/unplug-whitelist
Commit: ed78b557cd078f6d7cfd402d37ad167e191bd706
Parents: 83377d3
Author: Ian Clelland <ic...@chromium.org>
Authored: Wed Oct 22 16:27:28 2014 -0400
Committer: Ian Clelland <ic...@chromium.org>
Committed: Thu Oct 30 12:19:06 2014 -0400
----------------------------------------------------------------------
.../src/org/apache/cordova/ConfigXmlParser.java | 35 --------------------
1 file changed, 35 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/cordova-android/blob/ed78b557/framework/src/org/apache/cordova/ConfigXmlParser.java
----------------------------------------------------------------------
diff --git a/framework/src/org/apache/cordova/ConfigXmlParser.java b/framework/src/org/apache/cordova/ConfigXmlParser.java
index 5dd9ff8..1ea30b0 100644
--- a/framework/src/org/apache/cordova/ConfigXmlParser.java
+++ b/framework/src/org/apache/cordova/ConfigXmlParser.java
@@ -36,18 +36,8 @@ public class ConfigXmlParser {
private String launchUrl = "file:///android_asset/www/index.html";
private CordovaPreferences prefs = new CordovaPreferences();
- private Whitelist internalWhitelist = new Whitelist();
- private Whitelist externalWhitelist = new Whitelist();
private ArrayList<PluginEntry> pluginEntries = new ArrayList<PluginEntry>(20);
- public Whitelist getInternalWhitelist() {
- return internalWhitelist;
- }
-
- public Whitelist getExternalWhitelist() {
- return externalWhitelist;
- }
-
public CordovaPreferences getPreferences() {
return prefs;
}
@@ -81,11 +71,6 @@ public class ConfigXmlParser {
public void parse(XmlResourceParser xml) {
int eventType = -1;
- // Add implicitly allowed URLs
- internalWhitelist.addWhiteListEntry("file:///*", false);
- internalWhitelist.addWhiteListEntry("content:///*", false);
- internalWhitelist.addWhiteListEntry("data:*", false);
-
while (eventType != XmlResourceParser.END_DOCUMENT) {
if (eventType == XmlResourceParser.START_TAG) {
handleStartTag(xml);
@@ -121,26 +106,6 @@ public class ConfigXmlParser {
else if (paramType.equals("onload"))
onload = "true".equals(xml.getAttributeValue(null, "value"));
}
- else if (strNode.equals("access")) {
- String origin = xml.getAttributeValue(null, "origin");
- String subdomains = xml.getAttributeValue(null, "subdomains");
- boolean external = (xml.getAttributeValue(null, "launch-external") != null);
- if (origin != null) {
- if (external) {
- externalWhitelist.addWhiteListEntry(origin, (subdomains != null) && (subdomains.compareToIgnoreCase("true") == 0));
- } else {
- if ("*".equals(origin)) {
- // Special-case * origin to mean http and https when used for internal
- // whitelist. This prevents external urls like sms: and geo: from being
- // handled internally.
- internalWhitelist.addWhiteListEntry("http://*/*", false);
- internalWhitelist.addWhiteListEntry("https://*/*", false);
- } else {
- internalWhitelist.addWhiteListEntry(origin, (subdomains != null) && (subdomains.compareToIgnoreCase("true") == 0));
- }
- }
- }
- }
else if (strNode.equals("preference")) {
String name = xml.getAttributeValue(null, "name").toLowerCase(Locale.ENGLISH);
String value = xml.getAttributeValue(null, "value");
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cordova.apache.org
For additional commands, e-mail: commits-help@cordova.apache.org