Download & Extend

New CAPTCHA submodule

Project:CAPTCHA
Version:7.x-1.0-beta2
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:needs work

Issue Summary

Hi,

I have created a graphical version of the math CAPTCHA based on the Image CAPTCHA submodule, which I propose to be made a part of the CAPTCHA module. It is easy to use, its configuration almost the same as the Image CAPTCHA's, and it is harder to crack than both Image CAPTCHA and the orig math CAPTCHA. An image of an example CAPTCHA is uploaded. If you are interested, please contact me.

Regards
Belaselm

AttachmentSizeStatusTest resultOperations
GraphMath_Captcha.png15.29 KBIgnored: Check issue status.NoneNone

Comments

#1

Good! Why don't you create a sandbox and apply to be a module?

#2

That's probably because I can't access the git repository. (I could, but it seems I did something wrong, and the repository refuses the connection.) Anyway, this captcha is almost identical in code to the image captcha, and the applying rules say that in this case I'm to contact the authors of the original module, and offer it as a contribution/patch.

#3

Then you can try to make a patch or upload a tarball first. However, git is a must in drupal development.

#4

Status:active» postponed (maintainer needs more info)

rogical, could you upload a patch?

#5

Status:postponed (maintainer needs more info)» active

Yes, I know, but I have just started using git, and I'm not perfect from it yet. For now, I'm attaching a tarball archive. I'll also have two more captchas soon, I'll try to make them as independent modules.

AttachmentSizeStatusTest resultOperations
graphmath_captcha.tar_.gz64.82 KBIgnored: Check issue status.NoneNone

#6

Tested, seems all well.

#7

Sorry, I have left some obsolete parts in the code, like UTF-8 split, which is unnecessary for numbers. Here is a modified version without these.

AttachmentSizeStatusTest resultOperations
graphmath_captcha_corrected.tgz63.82 KBIgnored: Check issue status.NoneNone

#8

New files seems can't to be made in patch.

#9

Sorry, as I wrote I'm still new in this business. Will this diff to the originally uploaded graphmath_captcha.tar_.gz file be enough?

AttachmentSizeStatusTest resultOperations
graphmath_captcha.diff2.4 KBIgnored: Check issue status.NoneNone

#10

I have created a sandbox project, and uploaded the files of this GraphMath CAPTCHA there. You can find it at http://drupalcode.org/sandbox/Belaselm/1676284.git/

#11

I have created a git patch from the proposed graphmath_captcha submodule. I hope it will be OK.

AttachmentSizeStatusTest resultOperations
captcha-7.x-1.0-beta2-1670896.patch44.08 KBIgnored: Check issue status.NoneNone

#12

Status:active» needs work

Hi,

sorry for the delayed answer. Looking at the patch of #11, I think there is way too much code duplication with the image CAPTCHA module. It would be pretty bad for long term maintainability to do it like this. Why didn't you reuse the functions from image CAPTCHA directly?

#13

Thanks for the suggestion, I'll work on it when I'll have some time. I thought about making this captcha a part of the image captcha module, along with a configuration option to select it, but if we keep it a submodule, it can be used alongside the image captcha.

#14

Gr8 work Belaselm. Captchas are pretty hard to code..

nobody click here