You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@roller.apache.org by GitBox <gi...@apache.org> on 2021/05/31 15:09:46 UTC

[GitHub] [roller] mbien commented on pull request #90: Parse referrer URL instead of using string value directly.

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


   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)
   


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