[GitHub] [camel-quarkus] ppalaga commented on issue #493: Issue 490: Add tagsoup support

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[GitHub] [camel-quarkus] ppalaga commented on issue #493: Issue 490: Add tagsoup support

GitBox
ppalaga commented on issue #493: Issue 490: Add tagsoup support
URL: https://github.com/apache/camel-quarkus/pull/493#issuecomment-560588791
 
 
   > I have pushed a fix but this just yields another error. I really do not know how to debug this last one.
   
   XPath is untested yet, so seeing it to fail in native mode is rather expected. But it is a good news actually, that the flow fails that late, because it means that TagSoup produces a valid DOM. I think the extension in itself works well, but we need to change the way we test it somehow.
   
   I tried to serialize the DOM using a dummy XSLT but it fails due to issues that need to get fixed in the XSLT extension.
   
   Then I tried to use the DOM API directly to return the content of the text node under /html/body/p and it works. I have sent a PR against your branch: https://github.com/jsight/camel-quarkus/pull/1 Please check whether it makes sense.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[hidden email]


With regards,
Apache Git Services