You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@flink.apache.org by rm...@apache.org on 2020/03/12 16:01:57 UTC
[flink] branch release-1.10 updated: [FLINK-16550][s3] Force
namespace parsing in XmlResponsesSaxParser.
This is an automated email from the ASF dual-hosted git repository.
rmetzger pushed a commit to branch release-1.10
in repository https://gitbox.apache.org/repos/asf/flink.git
The following commit(s) were added to refs/heads/release-1.10 by this push:
new 15a149e [FLINK-16550][s3] Force namespace parsing in XmlResponsesSaxParser.
15a149e is described below
commit 15a149ebc4042c5f623c88c1afddb826bc4d09e7
Author: Arvid Heise <ar...@ververica.com>
AuthorDate: Wed Mar 11 22:37:42 2020 +0100
[FLINK-16550][s3] Force namespace parsing in XmlResponsesSaxParser.
This commit explicitly enables Sax' NAMESPACES_FEATURE (default true) in JAXP (default false), such that multi part uploads to S3 actually work.
---
.../amazonaws/services/s3/model/transform/XmlResponsesSaxParser.java | 2 ++
1 file changed, 2 insertions(+)
diff --git a/flink-filesystems/flink-s3-fs-base/src/main/java/com/amazonaws/services/s3/model/transform/XmlResponsesSaxParser.java b/flink-filesystems/flink-s3-fs-base/src/main/java/com/amazonaws/services/s3/model/transform/XmlResponsesSaxParser.java
index bc87b23..d6a3225 100644
--- a/flink-filesystems/flink-s3-fs-base/src/main/java/com/amazonaws/services/s3/model/transform/XmlResponsesSaxParser.java
+++ b/flink-filesystems/flink-s3-fs-base/src/main/java/com/amazonaws/services/s3/model/transform/XmlResponsesSaxParser.java
@@ -128,6 +128,8 @@ public class XmlResponsesSaxParser {
public XmlResponsesSaxParser() throws SdkClientException {
// Ensure we can load the XML Reader.
try {
+ // Need to explicitly enable Sax' NAMESPACES_FEATURE (default true) in JAXP (default false)!
+ SAX_PARSER_FACTORY.setNamespaceAware(true);
xr = SAX_PARSER_FACTORY.newSAXParser().getXMLReader();
disableExternalResourceFetching(xr);
} catch (SAXException | ParserConfigurationException e) {