search

codingteam / loglist

Reincarnation of the famous service
https://loglist.xyz/
MIT License
7 stars 4 forks source link

Add a way to disable ReCAPTCHA during development #149

Open Minoru opened 8 years ago

Minoru commented 8 years ago

It'd be nice to flip a toggle or something and be able to develop locally without worrying about ReCAPTCHA. As it is, I had to edit a couple of files to make Loglist believe that CAPTCHA check was passed successfully.

rexim commented 8 years ago

This is strange. ReCAPTCHA should work if your host is localhost.

ForNeVeR commented 8 years ago

Maybe he wanted to edit the files without an active internet connection? I think that's a valid point.

rexim commented 8 years ago

@Minoru did you, really?

rexim commented 8 years ago

Anyway, we have config for such things. Feel free to add an option.

Minoru commented 8 years ago

I have an active Internet connection. No, I didn't try to work on LogList without one. Still, ReCAPTCHA doesn't work, claiming I have invalid site key. The docs claim that "[b]y default, all keys work on "localhost" (or "127.0.0.1"), so you can always develop and test on your local machine.", but that doesn't seem to be the case. Any advice?

Minoru commented 8 years ago

JFTR, I didn't edit RECAPTCHA_{PUBLIC,PRIVATE}_KEY settings in devenv file. Should I?

rexim commented 8 years ago

Hah! Looks like Mr @ForNeVeR and I use the the same keys that are in production! :D

rexim commented 8 years ago

That's why everything work for us.

rexim commented 8 years ago

You can create your own RECAPTCHA keys if you want

Minoru commented 8 years ago

Yeah, so it turns out I should generate and specify valid keys in order for ReCAPTCHA to work. I'll update the README in the moment.

ForNeVeR commented 8 years ago

It seems that you need to obtain your own key even to use it on localhost. I think that we should add an option to disable CAPTCHA. It will be easier for our contributors to install local instances without registration and SMS, you know.

Minoru commented 8 years ago

I don't see a clear way to add an option, so the only thing I can do right now is clarify the docs. Will such a PR be welcome?

ForNeVeR commented 8 years ago

Yes, it will be good (but not enough). I guess someone else could later resolve the issue completely.

rexim commented 8 years ago

K, taking this one then

rexim commented 8 years ago

partially fixes

ForNeVeR commented 8 years ago

@Minoru, you should definitely be more careful with your words.

Aakash2408 commented 2 years ago

can i block Recaptcha on specific dev environments in my codebase?

Minoru commented 2 years ago

@Aakash2408 Currently you can't, we haven't implemented a configuration setting for that yet. Would you like to work on that?