[ 
https://issues.apache.org/jira/browse/NUTCH-567?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12535853
 ] 

Dawid Weiss commented on NUTCH-567:
-----------------------------------

Don't mention it. Happy birthday and I hope it'll work for you. If you take a 
look at the patch (source) you'll see it's really a trivial change to the 
source... I actually looked at how browsers handle such "illegal" URIs (because 
all URIs should have & in them to separate parameters, not just an 
ampersand) and it seems they use some heuristics to determine what is an entity 
and what is not. Look at the test case -- it shows such a nasty situation. The 
current patch attempts to resolve URIs in a similar way my Firefox does it.

> Proper (?) handling of URIs in TagSoup.
> ---------------------------------------
>
>                 Key: NUTCH-567
>                 URL: https://issues.apache.org/jira/browse/NUTCH-567
>             Project: Nutch
>          Issue Type: Improvement
>            Reporter: Dawid Weiss
>            Priority: Minor
>         Attachments: tagsoup-1.1.3-uripatched.jar , uri-entities.patch
>
>
> Doug Cook reported that TagSoup incorrectly handles some URI parameters. More 
> discussion on the list and at TagSoup's mailing list.
> http://tech.groups.yahoo.com/group/tagsoup-friends/message/838
> I looked at the sources of TagSoup because I'm using it myself (although the 
> URIs are not relevant for me). It seems like you can implement a naive 
> workaround by remembering the parsing state and just avoiding entity 
> resolution. Attached is the patch that does this.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to