When a user has an IPv6 address, the error message, 'Invalid hashcash (Additional checks fail) ' is given. This is because the hashcash value parts are separated by ':' and IPv6 addresses use ':' as a separator rather than a '.' as in IPv4 addresses.

CommentFileSizeAuthor
#2 hashcash-ipv6-fix-2600602-2.patch3.62 KBrgon

Comments

rgon created an issue. See original summary.

rgon’s picture

rgon commented
StatusFileSize
new hashcash-ipv6-fix-2600602-2.patch3.62 KB

Patch attached for 7.x-2.x branch.

I created a patch which updates hashcash.module and hashcash.js.inc to translate the IPv6 addresses to use '_' instead of ':' for the hashcash comparison.

rgon’s picture

rgon commented
Status: Needs work » Needs review
rgon’s picture

rgon commented
Issue summary: View changes
sdrycroft’s picture

sdrycroft commented

Thanks for this Ron, I am in the process of moving jobs, so may not be able to look into this for a little while.

rgon’s picture

rgon commented

Simon, do you have time to look into this now?

rgon’s picture

rgon commented

sdrycroft,

Could you please enable the testbot for this project?

Thanks.