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

Doğacan Güney commented on NUTCH-522:
-------------------------------------

> So, I guess we have to add a new check regarding space in URLs. any idea ?

OK, it is a bug then. 

I would suggest that you add a main method to UrlValidator ( like this one: 
http://www.ceng.metu.edu.tr/~e1345172/validator_main.patch ), then debug 
UrlValidator to check why it accepts it. Also, if commons-validator's 
UrlValidator filters that url, you can debug original UrlValidator to see where 
it invalidates it.

My guess is that it may be related to LEGAL_ASCII_PATTERN. I couldn't get 
original validator's LEGAL_ASCII_PATTERN to work with java.util.regex so I 
wrote a new pattern, but I thought the new pattern was stricter than the old 
one.

> Use URLValidator in the Injector
> --------------------------------
>
>                 Key: NUTCH-522
>                 URL: https://issues.apache.org/jira/browse/NUTCH-522
>             Project: Nutch
>          Issue Type: Improvement
>          Components: injector
>            Reporter: Emmanuel Joke
>            Priority: Minor
>             Fix For: 1.0.0
>
>         Attachments: NUTCH-522.patch, NUTCH-522_v2.patch
>
>
> Same as NUTCH-505, we should use the UrlValidator to check url in the Injector

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


-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Nutch-developers mailing list
Nutch-developers@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nutch-developers

Reply via email to