http://localhost/ticket/1313 <p> Awesome to have the captcha implementation and running on thedatahub.org! Here are some further suggestions concerning the config options, before this goes into a release: </p> <ul><li>Document "ckan.recaptcha.publickey" and "ckan.recaptcha.privatekey" in doc/configuration.rst </li><li>Which one config option defines whether recaptcha is enabled? There is disagreement between the template and the lib. Maybe better to have a third option for this. When setting the syntax for this, I can see us wanting to use captcha for other operations, like package edit, package create, group create etc. so perhaps we'd want to be able to turn each one on/off separately, using the name of the logic action. e.g. <tt>ckan.recaptcha.enable = user_create, group_create</tt> </li><li>We got this exception last night from the template: <pre class="wiki">Module ?:37 in &lt;Expression u'g.recaptcha_publickey'&gt; &lt;&lt; &lt;dd&gt;&lt;input type="password" name="password2" value="" /&gt;&lt;/dd&gt; &lt;dd py:if="g.recaptcha_publickey"&gt; &lt;script type="text/javascript" src="http://www.google.com/recaptcha/api/challenge?k=${g.recaptcha_publickey}"&gt; &gt;&gt; &lt;dd py:if="g.recaptcha_publickey"&gt; Module genshi.template.eval:324 in lookup_attr &lt;&lt; val = obj[key] except (KeyError, TypeError): val = cls.undefined(key, owner=obj) return val &gt;&gt; val = cls.undefined(key, owner=obj) Module genshi.template.eval:410 in undefined &lt;&lt; """Raise an ``UndefinedError`` immediately.""" __traceback_hide__ = True raise UndefinedError(key, owner=owner) &gt;&gt; raise UndefinedError(key, owner=owner) UndefinedError: &lt;ckan.lib.app_globals.Globals object at 0xb8f1182c&gt; has no member named "recaptcha_publickey" </pre></li></ul><p> I'm not sure why - might be something to do with the maintenance on the server, but worth double checking, since we don't want exceptions, say if one or other config variable is not defined. </p> en-us http://assets.okfn.org/p/ckan/img/ckan_logo_shortname.png http://localhost/ticket/1313 Trac 0.12.3 rgrp Mon, 26 Sep 2011 22:33:52 GMT http://localhost/ticket/1313#comment:1 http://localhost/ticket/1313#comment:1 <ul> <li><strong>status</strong> changed from <em>new</em> to <em>closed</em> </li> <li><strong>resolution</strong> set to <em>duplicate</em> </li> </ul> <p> Error was due to maintenance. Need both config options for recaptcha to work so either one can be used to check config. Re docs see <a class="assigned ticket" href="http://localhost/ticket/1358" title="enhancement: Generate configuration documentation automatically from the ... (assigned)">#1358</a> (auto generation of docs on config from the config source). </p> <p> Given that only docs remain and that is part of <a class="assigned ticket" href="http://localhost/ticket/1358" title="enhancement: Generate configuration documentation automatically from the ... (assigned)">#1358</a> am closing as duplicate. </p> Ticket