Kleijn36 Posted April 22, 2019 Posted April 22, 2019 The last few days i have a lot of unwanted customer registrations. It looks that there are some registration bots running, i blocked the IP's but stil new unwanted registrations. In the last name they register a www. domain, how to block this? Is there a solution too not accept "www." in the last name registration? 1
basix101 Posted April 22, 2019 Posted April 22, 2019 (edited) I have the same unwanted Customer Registrations from the same spammer. I'm not a coder, just a copy and paster! Can you please tell me which is the correct code to alter as I am confused? 1: https://github.com/thirtybees/thirtybees/commit/7ad147d991861e498e586f4dfc8ad1ff 2: https://github.com/thirtybees/thirtybees/commit/c733d5360d5292c12bb6a899748d9094bc608e05 3: https://github.com/thirtybees/thirtybees/commit/f04f1ded917b702465b2da6adfb7d573fdb974ad Thank you Edited April 22, 2019 by basix101 found another github link
DRMasterChief Posted April 22, 2019 Posted April 22, 2019 I think a good To-Do-List for this is necessary, @Traumflug maybe you can arrange some advice for this? Seems not to be that easy to find out what to do with these Github things, i have also some difficulties with it. 1
wakabayashi Posted April 22, 2019 Posted April 22, 2019 Its very easy to make github changes: Open the file (you see it beside the green and red sqaures. for example: classes/Validate.php) You remove all the red lines in the code You add all the green lines in the code. 1
DRMasterChief Posted April 22, 2019 Posted April 22, 2019 Yep, but in all 3 Github links is the same file which should be changed, but with different changes (classes/Validate.php) , think this is very strange for some people. Or am i wrong, hopefully not :) regarding i will do this changes in 1-2 days. btw. will there be an update for this (i think it will be included in the next, but not sure when this will come)? 1
colorful-ant Posted April 22, 2019 Posted April 22, 2019 i made backup/rename from old file (validate.php) - insert new file, all is good 1
basix101 Posted April 22, 2019 Posted April 22, 2019 OK, I think I got it. The two GitHub validate.php files looked different because one was an update of the other. Once I found the one that matched my own validate.php file and saw the 'updated' link to the other GitHub validate.php file it made sense. I also changed the validate.js files in my shop. Thanks for all your time and input, it is sincerely appreciated.
Guest Posted April 23, 2019 Posted April 23, 2019 (edited) 19 hours ago, DRMasterChief said: Yep, but in all 3 Github links is the same file which should be changed, but with different changes (classes/Validate.php) , think this is very strange for some people. Or am i wrong, hopefully not 🙂 regarding i will do this changes in 1-2 days. btw. will there be an update for this (i think it will be included in the next, but not sure when this will come)? I was getting hit with these too. Fix was really easy to implement. Thanks a lot for releasing this so quickly. You do all 3 of these. Do this one first:https://github.com/thirtybees/thirtybees/commit/c733d5360d5292c12bb6a899748d9094bc608e05 Then this one:https://github.com/thirtybees/thirtybees/commit/7ad147d991861e498e586f4dfc8ad1ff Then this one: (also edit the .js file here)https://github.com/thirtybees/thirtybees/commit/f04f1ded917b702465b2da6adfb7d573fdb974ad Edited April 23, 2019 by Purity
Guest Posted April 24, 2019 Posted April 24, 2019 (edited) The patch of course works, but it seems to also disable the phrase c/o, which is a very valid and common way to ship in the USA when you're shipping to an address that isn't yours. Although it still works in the company box. Edited April 24, 2019 by Purity
Traumflug Posted April 24, 2019 Posted April 24, 2019 You mean forward slashes as part of the name? Yes, they were disallowed to avoid names like "cat /etc/passwd". Looks like there's another commit needed. Dealing with names is a tricky matter, see https://shinesolutions.com/2018/01/08/falsehoods-programmers-believe-about-names-with-examples/
Traumflug Posted April 24, 2019 Posted April 24, 2019 Here we go: https://github.com/thirtybees/thirtybees/commit/1818e2aca9b90dbee0553272e45956b275fda334 2
vsn Posted April 24, 2019 Posted April 24, 2019 why you guys don't use a TB native module called reCaptcha? You can set up a reCaptcha validation during the registration process. I faced the same spam-registration problem, installed the module, since that no unwanted registration anymore.
wakabayashi Posted April 24, 2019 Posted April 24, 2019 I use the captcha module for contact form. But I wouldnt do that for registration. For me its too risky, that it fails or people cant manage it
Kleijn36 Posted April 25, 2019 Author Posted April 25, 2019 On 4/23/2019 at 5:44 PM, datakick said: also coreupdater can be used to fix this Manual fixed the problem in Validate.php. Coreupdater doesn't fix the problem, after update (1.0.8) problem is back.
Traumflug Posted April 26, 2019 Posted April 26, 2019 13 hours ago, Kleijn36 said: after update (1.0.8) problem is back @datakick probably means an update not to 1.0.8, but to Bleeding Edge / 1.0.x. That's currently pretty stable.
DRMasterChief Posted April 26, 2019 Posted April 26, 2019 On 4/24/2019 at 12:16 PM, Traumflug said: Here we go: https://github.com/thirtybees/thirtybees/commit/1818e2aca9b90dbee0553272e45956b275fda334 This should fix the c/o problem? It does not for me, i have used this php file and can not register with a c/o in the name.
DRMasterChief Posted March 9 Posted March 9 Would like to raise the topic, is this now implemented in an newer version of tb ? I cant find the changes from @Traumflug in the actual validate.php , but maybe this can be very helpful.
the.rampage.rado Posted March 9 Posted March 9 Those changes are included: If you get spam registrations install Or I'm using both, Cloudflare for front office forms and No Captcha for my BO.
DRMasterChief Posted March 9 Posted March 9 hmm, ok thank you, but i can not find these changes in my validate.php Cloudflare Turnstile is the free version? Is it a good choice?
the.rampage.rado Posted March 9 Posted March 9 I have them, so probably you're not looking for the proper strings or your install is ages old. https://github.com/eschiendorfer/genzo_turnstile , thanks to @wakabayashi 2
DRMasterChief Posted March 9 Posted March 9 with the update to 1.5.1 a few minutes ago i now have the 2 acutal validate files , Notepad++ says 🙂
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now