[ https://issues.apache.org/jira/browse/CAMEL-12415?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16422819#comment-16422819 ]
ASF GitHub Bot commented on CAMEL-12415: ---------------------------------------- aldettinger commented on a change in pull request #2277: CAMEL-12415 - camel-jaxb, fix options combination: encoding,filterNon… URL: https://github.com/apache/camel/pull/2277#discussion_r178595038 ########## File path: components/camel-jaxb/src/main/java/org/apache/camel/converter/jaxb/JaxbDataFormat.java ########## @@ -206,9 +206,10 @@ void doMarshal(Exchange exchange, Object graph, OutputStream stream, Marshaller // only marshal if its possible if (introspector.isElement(element)) { if (asXmlStreamWriter(exchange)) { - XMLStreamWriter writer = typeConverter.convertTo(XMLStreamWriter.class, stream); + XMLStreamWriter writer = typeConverter.convertTo(XMLStreamWriter.class, exchange, stream); if (needFiltering(exchange)) { - writer = new FilteringXmlStreamWriter(writer); + String charset = exchange.getProperty(Exchange.CHARSET_NAME, String.class); Review comment: Could you please change the method signature to `doMarshal(..., OutputStream stream, String charset, ...)` and get the charset from there ? ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Camel-jaxb option "encoding" with option "filterNonXmlChars" generate wrong > data > -------------------------------------------------------------------------------- > > Key: CAMEL-12415 > URL: https://issues.apache.org/jira/browse/CAMEL-12415 > Project: Camel > Issue Type: Bug > Components: camel-jaxb > Affects Versions: 2.21.0 > Environment: OS X 13.3, Java 8 > Reporter: Jonas Waage > Priority: Minor > > When using the jaxb-component to marshal. > The properties: > * Encoding > * FilterNonXmlChars > do not work together correctly. > FilterNonXmlChars will ignore the set encoding, and make the output bytes > UTF-8 encoded. > I would either expect this to just work, and by that I mean bytes should be > encoded as the set encoding, or minimally fail during startup of the route > with an exception explaining that these properties does not work together. > I'd really prefer the first, since I want to use the functionality of both. > I have provided a patch which makes this work.I have done a small refactoring > to not have to duplicate more code. I can rewrite this if it is a problem. > Below is a test which will reproduce the problem. > {code:java} > public class ExplicitEncodingAndXMLCharFilteringTest extends CamelTestSupport > { > @Override > public void setUp() throws Exception { > deleteDirectory("target/charset"); > super.setUp(); > } > @Test > public void testIsoAndCharacterFiltering() throws Exception { > PurchaseOrder order = new PurchaseOrder(); > //Data containing characters ÆØÅæøå that differ in utf-8 and iso + a > spouting whale > String name = > "\u00c6\u00d8\u00C5\u00e6\u00f8\u00e5\uD83D\uDC33\uFFFD"; > String expected = "\u00c6\u00d8\u00C5\u00e6\u00f8\u00e5 \uFFFD"; > //Spouting whale has become spaces > order.setName(name); > order.setAmount(123.45); > order.setPrice(2.22); > MockEndpoint result = getMockEndpoint("mock:file"); > result.expectedFileExists("target/charset/output.xml"); > template.sendBody("direct:start", order); > assertMockEndpointsSatisfied(); > JAXBContext jaxbContext = > JAXBContext.newInstance("org.apache.camel.example"); > Unmarshaller unmarshaller = jaxbContext.createUnmarshaller(); > InputStream inputStream = new > FileInputStream("target/charset/output.xml"); > Reader reader = new InputStreamReader(inputStream, "ISO-8859-1"); > PurchaseOrder obj = (PurchaseOrder) unmarshaller.unmarshal(reader); > assertEquals(expected, obj.getName()); > } > @Override > protected RouteBuilder createRouteBuilder() throws Exception { > return new RouteBuilder() { > @Override > public void configure() throws Exception { > JaxbDataFormat jaxb = new > JaxbDataFormat("org.apache.camel.example"); > jaxb.setFilterNonXmlChars(true); > jaxb.setEncoding("iso-8859-1"); > from("direct:start") > .marshal(jaxb) > > .to("file:target/charset/?fileName=output.xml&charset=iso-8859-1"); > } > }; > } > {code} > -- This message was sent by Atlassian JIRA (v7.6.3#76005)