> On 2012-01-04 19:48:10, Marlon Pierce wrote: > > https://svn.apache.org/repos/asf/incubator/rave/trunk/rave-components/rave-core/pom.xml, > > line 144 > > <https://reviews.apache.org/r/3380/diff/1/?file=66372#file66372line144> > > > > Do you know the licensing situation with this jar? This will need to > > added to the NOTICE files (./rave-portal/src/main/dist/NOTICE) assuming it > > is Apache V2 compatible. > > marijan milicevic wrote: > It is Apache V2 licensed, see: http://code.google.com/p/recaptcha4j/
recaptcha4j is Apache v2 Licensed. It doesn't have its own NOTICE file but the project should be mentioned ineeed. http://code.google.com/p/recaptcha4j/ > On 2012-01-04 19:48:10, Marlon Pierce wrote: > > https://svn.apache.org/repos/asf/incubator/rave/trunk/rave-components/rave-core/src/test/resources/portal.properties, > > line 41 > > <https://reviews.apache.org/r/3380/diff/1/?file=66375#file66375line41> > > > > Should be placed earlier for alphabetical ordering. > > marijan milicevic wrote: > ok The portal.properties aren't ordered alphabetically. Personally I prefer "global" settings in the top and settings for a specific functionality to the bottom. > On 2012-01-04 19:48:10, Marlon Pierce wrote: > > https://svn.apache.org/repos/asf/incubator/rave/trunk/rave-portal-resources/src/main/resources/portal.properties, > > line 57 > > <https://reviews.apache.org/r/3380/diff/1/?file=66378#file66378line57> > > > > I'm not sure what to put here to enable this. Also, the error messages > > displayed on the new account page if you don't have > > portal.captcha.key.public and .private set correctly are too small and > > unobtrusive. > > marijan milicevic wrote: > You need to register at http://www.google.com/recaptcha > We should add documentation how to enable reCAPTCHA. You have to sign up for an API key. Any API key works on localhost. > On 2012-01-04 19:48:10, Marlon Pierce wrote: > > https://svn.apache.org/repos/asf/incubator/rave/trunk/rave-portal-resources/src/main/webapp/WEB-INF/jsp/views/newaccount.jsp, > > line 83 > > <https://reviews.apache.org/r/3380/diff/1/?file=66379#file66379line83> > > > > I assume this outputs the error message for misconfigured properties. > > If so, the error messages are easily missed. Need to have an appropriate > > styling. > > marijan milicevic wrote: > I'll see if I can intercept invalid configuration & display "better" > error messages This fieldset will contain the script that renders the reCAPTCHA - Jasha ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3380/#review4187 ----------------------------------------------------------- On 2012-01-04 16:08:40, marijan milicevic wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/3380/ > ----------------------------------------------------------- > > (Updated 2012-01-04 16:08:40) > > > Review request for rave. > > > Summary > ------- > > ReCaptcha implementation > > > Diffs > ----- > > https://svn.apache.org/repos/asf/incubator/rave/trunk/pom.xml 1227203 > > https://svn.apache.org/repos/asf/incubator/rave/trunk/rave-components/rave-core/pom.xml > 1227203 > > https://svn.apache.org/repos/asf/incubator/rave/trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/service/CaptchaService.java > PRE-CREATION > > https://svn.apache.org/repos/asf/incubator/rave/trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/service/impl/ReCaptchaService.java > PRE-CREATION > > https://svn.apache.org/repos/asf/incubator/rave/trunk/rave-components/rave-core/src/test/resources/portal.properties > 1227203 > > https://svn.apache.org/repos/asf/incubator/rave/trunk/rave-components/rave-web/src/main/java/org/apache/rave/portal/web/controller/NewAccountController.java > 1227203 > > https://svn.apache.org/repos/asf/incubator/rave/trunk/rave-components/rave-web/src/test/java/org/apache/rave/portal/web/controller/NewAccountControllerTest.java > 1227203 > > https://svn.apache.org/repos/asf/incubator/rave/trunk/rave-portal-resources/src/main/resources/portal.properties > 1227203 > > https://svn.apache.org/repos/asf/incubator/rave/trunk/rave-portal-resources/src/main/webapp/WEB-INF/jsp/views/newaccount.jsp > 1227203 > > https://svn.apache.org/repos/asf/incubator/rave/trunk/rave-portal/src/test/resources/portal.properties > 1227203 > > Diff: https://reviews.apache.org/r/3380/diff > > > Testing > ------- > > > Thanks, > > marijan > >
