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) {