Home » Download » Modules » Hashcash » Issues

Hashcash updated for Drupal 6.x - code review needed

rick_deckard - September 22, 2008 - 01:00
Project:Hashcash
Version:5.x-1.4
Component:Code
Category:support request
Priority:normal
Assigned:Unassigned
Status:closed
Description

I just created an updated version of Hashcash and got rid of all minor and major notices issued by the coder module.

I also updated the intall to use the new hook_schema() intall method.

It seems to be working on my site just fine (no crashes). It will take a few days to see if it stops the spam flood, but a hashcash plugin on Wordpress sites I've done just killed spam pretty much right away.

Here is a zip archive of the 6.x module. Sorry it's not a true patch, but you should be able to just do a diff locally and generate a patch.

AttachmentSize
hashcash.zip11.46 KB
» Login or register to post comments

#1

rick_deckard - September 28, 2008 - 00:47

Forget it - many problems with that code. Some fixed, but still having trouble actually getting the onsubmit event to fire for some reason.

Login or register to post comments

#2

rick_deckard - September 28, 2008 - 04:51

Okay I think I've got it working now. Sorry about that first one.

AttachmentSize
hashcash-6.x-1.4-alpha.zip 10.88 KB
Login or register to post comments

#3

grendzy - January 2, 2009 - 17:27

subscribing, thanks for the patch. I'll try to review it soon. I'm interested in this module as an approach to #117056: Protection against brute force login.

Login or register to post comments

#4

grendzy - January 14, 2009 - 23:44

Your updated module works great for me! I have a couple of minor suggestions:

  • LICENSE.txt isn't needed (actually I think it will be blocked by Drupal CVS servers).
  • Also, the "version", "project", and "datestamp" lines should be omitted from the .info file (they will be added by drupal.org).
  • There was an extra "onsubmit", though commented out
  • Default form IDs don't match D6 (should be "user-login" and "user-login-form")
  • As long as you're making style tweaks, there are several more formatting changes suggested by coder in "minor" mode (trailing spaces, etc)

On the whole, a well done port. Thanks!

Login or register to post comments

#5

sdrycroft - February 4, 2009 - 13:06
Status:needs review» fixed

A D6 version has been released. The official version fixes the cache issue previously mentioned on this module's front page.

Login or register to post comments

#6

rick_deckard - February 11, 2009 - 20:17

Thanks for the notes grendzy and thanks even more sdrycroft for doing a real D6 version.

Login or register to post comments

#7

System Message - February 25, 2009 - 20:20
Status:fixed» closed

Automatically closed -- issue fixed for 2 weeks with no activity.

Login or register to post comments
 
 

Drupal is a registered trademark of Dries Buytaert.