5
11
Fork
You've already forked tomato
1

Add honeypot to stop spam submissions #166

Merged
edwardloveall merged 2 commits from el-honeypot into main 2026年06月08日 08:10:51 +02:00

We've had one spam submission so far. This may not stop every bot, but has the
ability to stop some.

I think ideally we should have an Onboarding model (not database backed) that
has all the organization fields and also the honeypot field. A couple of things
in the controller hint at wanting that (adding address/subscription/user
defaults, separate params for the honeypot, manual validation, etc). This still
felt like the best, simple system for now.

We've had one spam submission so far. This may not stop every bot, but has the ability to stop some. I think ideally we should have an Onboarding model (not database backed) that has all the organization fields and also the honeypot field. A couple of things in the controller hint at wanting that (adding address/subscription/user defaults, separate params for the honeypot, manual validation, etc). This still felt like the best, simple system for now.
Add honeypot to stop spam submissions
Some checks failed
ci/woodpecker/pr/ci Pipeline failed
535616ef49
We've had one spam submission so far. This may not stop every bot, but has the
ability to stop some.
I think ideally we should have an Onboarding model (not database backed) that
has all the organization fields and also the honeypot field. A couple of things
in the controller hint at wanting that (adding address/subscription/user
defaults, separate params for the honeypot, manual validation, etc). This still
felt like the best, simple system for now.
jsilasbailey left a comment
Copy link

🍯

🍯
edwardloveall force-pushed el-honeypot from 535616ef49
Some checks failed
ci/woodpecker/pr/ci Pipeline failed
to 004c703e3b
Some checks failed
ci/woodpecker/pr/ci Pipeline failed
2026年06月08日 07:33:09 +02:00
Compare
Time-of-day dependent spec
All checks were successful
ci/woodpecker/pr/ci Pipeline was successful
63375ab3e8
I happened to be up in the middle of the night running tests, and I found this
spec that failed. It set up a recurring event at 12pm and then looked for
occurrences in a two week range. But since I'm running this at like 1am, the
time range didn't quite get to the second event in the series.
Basing the time range from the event `starts_at` should make it work at any time
of day.
Sign in to join this conversation.
No reviewers
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
rootable/tomato!166
Reference in a new issue
rootable/tomato
No description provided.
Delete branch "el-honeypot"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?