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

Undefined array key "honey_time" again #59

Closed
v1ncen opened this issue May 19, 2022 · 4 comments
Closed

Undefined array key "honey_time" again #59

v1ncen opened this issue May 19, 2022 · 4 comments

Comments

@v1ncen
Copy link

v1ncen commented May 19, 2022

Sometimes I get Undefined array key "honey_time" errors on production.
It happens only some times when it's a spammer that tries to submit our form. Not all spammers are triggering the error. Non-spammer submissions are working fine.
Seems to me that some spammers found a way to remove 'honey_time' key from form payload.

I am using:

  • Laravel 9.8.1
  • php 8.1.5
  • Honey 0.4.0 (without any custom configuration and no livewire)
# My route:
Route::post('contact-us', 'ContactFormController@post')
    ->middleware(['honey'])
    ->name('contact.us');
# My form
<form action="{{ route('contact.us') }}" method="post">
    @csrf
    <x-honey/>
    ...
@AntonioJABV
Copy link

The same thing is happening to me. When a spammer submits the form without the honey_time field, I get a 500 error instead of denying the request.

I have created a PR to try to solve it: #60

@encodiaweb
Copy link

Same problem here. Is this PR mergeable?
Thank you

@AntonioJABV
Copy link

Hi @encodiaweb, technically the PR is ready to be merged, it's just pending of approval 😉 in the meantime, in our team, we are using the version of my fork until the merge is completed.

@lukeraymonddowning
Copy link
Owner

Thanks for the fix @AntonioJABV. And sorry for the very long delay 😛

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

No branches or pull requests

4 participants