This is an automated email from the ASF dual-hosted git repository. coheigea pushed a commit to branch 3.3.x-fixes in repository https://gitbox.apache.org/repos/asf/cxf.git
commit 41c834aea9bc6a537f7b336e626d07036e40b5c6 Author: Adrian Leonhard <adrianleonh...@gmail.com> AuthorDate: Tue Jan 14 13:35:05 2020 +0100 getBytes uses the platform charset by default. (#624) * getBytes uses the platform charset by default. As far as I can see, the only place where the schemas are loaded into the cache is loadSchemasIntoCache, where they are extracted from an XML Document. As such, they won't have an associated encoding. The XMLSource defaults to UTF-8, so this blows up if you have special characters and, for example, windows-1252 as the platform charset. * Alphabet hard * Update WadlGenerator.java https://www.joelonsoftware.com/2003/10/08/the-absolute-minimum-every-software-developer-absolutely-positively-must-know-about-unicode-and-character-sets-no-excuses/ * Remove BOS import (cherry picked from commit 543521ab051cb4f199faeab5765d001920bbbba2) --- .../java/org/apache/cxf/jaxrs/model/wadl/WadlGenerator.java | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/rt/rs/description/src/main/java/org/apache/cxf/jaxrs/model/wadl/WadlGenerator.java b/rt/rs/description/src/main/java/org/apache/cxf/jaxrs/model/wadl/WadlGenerator.java index 3573a14..1827e81 100644 --- a/rt/rs/description/src/main/java/org/apache/cxf/jaxrs/model/wadl/WadlGenerator.java +++ b/rt/rs/description/src/main/java/org/apache/cxf/jaxrs/model/wadl/WadlGenerator.java @@ -19,7 +19,6 @@ package org.apache.cxf.jaxrs.model.wadl; import java.io.ByteArrayInputStream; -import java.io.ByteArrayOutputStream; import java.io.IOException; import java.io.InputStream; import java.io.StringReader; @@ -32,6 +31,7 @@ import java.lang.reflect.Type; import java.lang.reflect.TypeVariable; import java.net.URI; import java.net.URL; +import java.nio.charset.StandardCharsets; import java.util.ArrayList; import java.util.Arrays; import java.util.Collection; @@ -1978,7 +1978,7 @@ public class WadlGenerator implements ContainerRequestFilter { this.theSchemas = new LinkedList<>(); // we'll need to do the proper schema caching eventually for (String s : schemas) { - XMLSource source = new XMLSource(new ByteArrayInputStream(s.getBytes())); + XMLSource source = new XMLSource(new ByteArrayInputStream(s.getBytes(StandardCharsets.UTF_8))); source.setBuffering(); Map<String, String> locs = getLocationsMap(source, "import", links, ui); locs.putAll(getLocationsMap(source, "include", links, ui)); @@ -2026,17 +2026,16 @@ public class WadlGenerator implements ContainerRequestFilter { } private String transformSchema(String schema, Map<String, String> locs) { - ByteArrayOutputStream bos = new ByteArrayOutputStream(); - SchemaConverter sc = new SchemaConverter(StaxUtils.createXMLStreamWriter(bos), locs); + StringWriter sw = new StringWriter(); + SchemaConverter sc = new SchemaConverter(StaxUtils.createXMLStreamWriter(sw), locs); try { StaxUtils.copy(new StreamSource(new StringReader(schema)), sc); sc.flush(); sc.close(); - return bos.toString(); + return sw.toString(); } catch (Exception ex) { return schema; } - } @Override