snoopdave commented on pull request #90:
URL: https://github.com/apache/roller/pull/90#issuecomment-855408372


   > LGTM, small remark for your consideration:
   > 
   > org.apache.commons.validator.routines.UrlValidator might be better since 
it does not rely on the exception in the program flow. Lib is already in the 
dependencies.
   > 
   > example snippet in 
[doc](https://commons.apache.org/proper/commons-validator/apidocs/org/apache/commons/validator/routines/UrlValidator.html)
   
   Thanks for the review. I switched over to `URLValidator` as you suggested.


-- 
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:
us...@infra.apache.org


Reply via email to