×

Notice

The forum is in read only mode.
Welcome, Guest
Username: Password: Remember me
  • Page:
  • 1

TOPIC: Recaptcha v2 Hotspots

Recaptcha v2 Hotspots 7 years 4 months ago #35259

  • Peter Ashford
  • Peter Ashford's Avatar Topic Author
  • Offline
  • Gold Boarder
  • Gold Boarder
  • Posts: 217
  • Thank you received: 33
Hello - The contact hotspot owner feature still links to V1 of recaptcha. Can this be changed? Would like to turn this on for users not logged in. If I try it, it fails unless I switch my recaptch plugin to use V1. I see the create hotspot has V2, but the email function still uses V1.

thanks Pete

Recaptcha v2 Hotspots 7 years 4 months ago #35264

  • Daniel Dimitrov
  • Daniel Dimitrov's Avatar
  • Offline
  • Administrator
  • Administrator
  • Posts: 9618
  • Karma: 155
  • Thank you received: 1081
I'll have to fix the code there. I forgot that we have recaptcha there.

Recaptcha v2 Hotspots 7 years 4 months ago #35265

  • Peter Ashford
  • Peter Ashford's Avatar Topic Author
  • Offline
  • Gold Boarder
  • Gold Boarder
  • Posts: 217
  • Thank you received: 33
Okay, thanks Daniel. Let me know when it is ready and I'll test. thanks Pete

Recaptcha v2 Hotspots 7 years 4 months ago #35267

  • Peter Ashford
  • Peter Ashford's Avatar Topic Author
  • Offline
  • Gold Boarder
  • Gold Boarder
  • Posts: 217
  • Thank you received: 33
Hi Daniel - When do you think it will be fixed? thanks Pete

Recaptcha v2 Hotspots 7 years 4 months ago #35276

  • Jean Machuron
  • Jean Machuron's Avatar
  • Offline
  • Junior Boarder
  • Junior Boarder
  • Posts: 35
  • Karma: 1
  • Thank you received: 0
mmmm just spent a couple of hours on this...
Subscribed to this post to also know the date of correction :-)
But for now <div class="g-recaptcha" data-sitekey="x"></div> instead of <div id="dynamic_recaptcha_1"></div> on tmpl hotspot/default.php solves it

Recaptcha v2 Hotspots 7 years 4 months ago #35277

  • Peter Ashford
  • Peter Ashford's Avatar Topic Author
  • Offline
  • Gold Boarder
  • Gold Boarder
  • Posts: 217
  • Thank you received: 33
Nice one Jean, I'll give that a try later.

Recaptcha v2 Hotspots 7 years 4 months ago #35283

  • Peter Ashford
  • Peter Ashford's Avatar Topic Author
  • Offline
  • Gold Boarder
  • Gold Boarder
  • Posts: 217
  • Thank you received: 33
That worked for me on my test site. Do you know if there are any security risks with hard coding the sitekey?

Recaptcha v2 Hotspots 7 years 4 months ago #35284

  • Jean Machuron
  • Jean Machuron's Avatar
  • Offline
  • Junior Boarder
  • Junior Boarder
  • Posts: 35
  • Karma: 1
  • Thank you received: 0
I guess that as long as this is the public key, there is no problem and Daniel should fix it soon anyway

Recaptcha v2 Hotspots 7 years 4 months ago #35285

  • Daniel Dimitrov
  • Daniel Dimitrov's Avatar
  • Offline
  • Administrator
  • Administrator
  • Posts: 9618
  • Karma: 155
  • Thank you received: 1081
Okay, released 5.3.3 with a fix for this.

@Peter there is no issue with displaying the site-key - it's there for exactly this purpose - to display it to everyone. What you shouldn't display is the secret key.

@Jean -> there is a better way to display the captcha now:
							<?php
							$captcha = JCaptcha::getInstance(HotspotsHelperSettings::get('captcha'));
						echo $captcha->display('compojoom-captcha', 'compojoom-contact-captcha', 'g-recaptcha');
							?>

With this code you should be able to use any captcha plugin and not just google's recaptcha.

Cheers,
Daniel
  • Page:
  • 1
Time to create page: 0.126 seconds