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

SQL syntax error in apache error.log #110

Open
than-sad opened this issue Oct 22, 2019 · 17 comments
Open

SQL syntax error in apache error.log #110

than-sad opened this issue Oct 22, 2019 · 17 comments
Assignees

Comments

@than-sad
Copy link

Error You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'WHERE active = 0 AND user_login = 'janessaxxxx' ORDER BY signup_id DESC LIMIT' at line 1 de la base de datos de WordPress para la consulta SELECT * FROM WHERE active = 0 AND user_login = 'janessaxxxx' ORDER BY signup_id DESC LIMIT 0, 1 realizada por require('wp-load.php'), require_once('wp-config.php'), require_once('wp-settings.php'), do_action('init'), WP_Hook->do_action, WP_Hook->apply_filters, ss_init, ss_check_white, be_load, ss_check_white->process, be_load, chkadminlog->process, wp_authenticate, apply_filters('authenticate'), WP_Hook->apply_filters, bp_core_signup_disable_inactive, BP_Signup::get, referer: https://www.xxxxx.es/wp-login.php

@bhadaway bhadaway added the bug label Oct 22, 2019
@bhadaway
Copy link
Owner

What version of Apache are you running?

@than-sad
Copy link
Author

than-sad commented Oct 23, 2019

7.3.10. Disabling Stop Spammers error disappears.

@than-sad
Copy link
Author

I tried with php 7.3.8 and 7.3.10 with same result. This error appears every time user try to login with non existing or incorrect username and password.

@bhadaway
Copy link
Owner

Ah, that's a PHP error and that's your PHP version, not Apache.

I haven't been able to recreate this error yet.

What version of WordPress, Stop Spammers, and MySQL are you on? Are you able to update those?

@than-sad
Copy link
Author

Yes sorry, it’s a Apache error log message, not Apache error. WP 5.2.4, SS 2019.1, MSQL 8.0.17. I have to check if there is an update for MSQL.

@than-sad than-sad changed the title Apache error Apache error log PHP error. Oct 23, 2019
@than-sad than-sad changed the title Apache error log PHP error. Apache error log PHP error message. Oct 23, 2019
@than-sad
Copy link
Author

I see there is a MySQL version 8.0.20. Should I try it?

@bhadaway
Copy link
Owner

No, everything is good on your end. I'm pretty sure the issue is MySQL 8, which Stop Spammer's code has not been tested or updated for.

I'll try and dig into this soon, but I've also put up the "Help Wanted" sign, in case anyone in the community can assist.

Thanks

@than-sad
Copy link
Author

Thanks. Anyway isn’t a critical error, just annoying to see repeated messages always in error log.

Few weeks ago I found information about this error, but now I can’t locate it. I will try again and post link here to give more information about this error.
Best regards!

@than-sad than-sad changed the title Apache error log PHP error message. SQL syntax error in apache error.log Oct 23, 2019
@StrothiP
Copy link

Hey guys,

I think I'm seeing the same issue on my server:

WordPress database error You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near ‘WHERE active = 0 AND user_login = ‘admin’ ORDER BY signup_id DESC LIMIT 0, 1′ at line 1 for query SELECT * FROM WHERE active = 0 AND user_login = ‘admin’ ORDER BY signup_id DESC LIMIT 0, 1 made by require(‘wp-load.php’), require_once(‘wp-config.php’), require_once(‘wp-settings.php’), do_action(‘init’), WP_Hook->do_action, WP_Hook->apply_filters, ss_init, ss_check_white, be_load, ss_check_white->process, be_load, chkadminlog->process, wp_authenticate, apply_filters(‘authenticate’), WP_Hook->apply_filters, bp_core_signup_disable_inactive, BP_Signup::get

@bhadaway Did you have a chance to look into what might be causing this?

Kind regards, Philip

@bhadaway
Copy link
Owner

Yes I did.

Fortunately, it doesn't affect the core plugin in any way. This issue is isolated to the beta features, database cleanup, and the vulnerability scan.

And luckily, it's just a simple matter of updating the syntax in the code to be MySQL 8 compatible. This fix will probably happen in the near future.

@than-sad
Copy link
Author

than-sad commented May 7, 2020

Hi, after 7 months my apache error log is full of this error. Nothing changed in all new releases...

Are you going to fix it or not?

Regards.

@than-sad
Copy link
Author

It´s ok, leave it. I found a better plugin.

@bhadaway
Copy link
Owner

Sorry, I was no longer involved with the project for a while, but I'm now working on it again.

Investigating...

@than-sad
Copy link
Author

than-sad commented Jun 12, 2020

One year and 8 months after I reported the error is obviously too late.

Im happily using Zero-Spam with no errors and no more spam registrations in buddy press from months ago.

Best regards.

@bhadaway
Copy link
Owner

Always sorry to lose a user, but I understand. It was just me for a few years, and now there's only two of us. We really don't have the kind of resources needed to push updates and fixes at the rate of some of the bigger plugins that make a lot of money with full dev teams, but we're trying to bring in revenue now, and we'll keep at it. Better later than never, for those still using Stop Spammers.

@bhadaway
Copy link
Owner

bhadaway commented Jun 12, 2020

The original author of Stop Spammers had simply disabled the vulnerability scanner altogether because it was incomplete and beta at best.

It's really hard, basically impossible to know what features are used and not used by Stop Spammers users, because we'd never be able to get enough feedback.

It's possible we might just need to remove this feature altogether, if it's too problematic.

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

4 participants