Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bulletproofing accessibility #5

Open
cferdinandi opened this issue Mar 24, 2016 · 2 comments
Open

Bulletproofing accessibility #5

cferdinandi opened this issue Mar 24, 2016 · 2 comments

Comments

@cferdinandi
Copy link
Collaborator

Currently, you're using some lightweight JS to remove the honeypot for human users. I see a couple of potential issues with the current implementation:

  1. Some bots can and do run JS.
  2. If JavaScript fails, the honeypot remains.
  3. If CSS fails, too, the honeypot is exposed, but there's nothing indicating to humans that they shouldn't fill it out.

I understand this is a crazy edge case, but one recommended approach you could take is to remove the JS bit that removes the honeypot, so that the honeypot field is always displayed. Then, apply a label to the honeypot that says something like, "If you're human, leave this blank."

If this sounds interesting and you'd like me to take a crack at it, let me know and I'll issue a PR.

@freak3dot
Copy link
Owner

I agree that there are issues with the current method. My concern would be that any indicator could also server as a flag to a bot.

  • Perhaps a random css class?
    ** Include it in the head of the document so it isn't close enough to the honeypot for bot to pick out.
  • Current JS solution would remain.

@cferdinandi
Copy link
Collaborator Author

My concern would be that any indicator could also server as a flag to a bot.

I shared that concern, but from my reading, this seems to maybe not matter as much? I'm by no means an expert on this, though, so whatever you think makes the most sense.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants