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

Recaptcha disappear on logout #36

Open
claudiosoprano opened this issue Sep 29, 2020 · 13 comments
Open

Recaptcha disappear on logout #36

claudiosoprano opened this issue Sep 29, 2020 · 13 comments

Comments

@claudiosoprano
Copy link

Hi all,

i have a problem with the rcguard plugin for roundcube.

When i logout from Roundcube, in the login page the Recaptcha window is not present, so if i try to login, it tells that i didn't use the recaptcha and then the recaptcha appear on the second login screen.

The only way to fix it, is reloading the page after the logout or insert the credentials twice (the first one with error).

I moved from v1.0.x to 1.1.x to 1.2.x to 1.3.x and finally 1.4.x the error is present on all the versions.

I don't know if there is a way to reload the page automatically on logout, let me know.

Thanks for the plugin.
Claudio Soprano

@claudiosoprano
Copy link
Author

just to show you what appear i will add two screenshoots

normal login screen
Screen Shot 2020-09-29 at 08 59 59

login screen after logout
Screen Shot 2020-09-29 at 08 59 39

Claudio

@dsoares
Copy link
Owner

dsoares commented Nov 28, 2020

Hi,
I can't reproduce your problem. I login, then logout and the recaptcha is shown in the login screen. Tested with recaptcha v2, larry and elastic skins, with $rcmail_config['failed_attempts'] = 1. What is your 'recaptcha_whitelist' config value?
Do you have more plugins enabled that could interfere with template_object_loginform hook?

@claudiosoprano
Copy link
Author

claudiosoprano commented Nov 30, 2020 via email

@smefa
Copy link

smefa commented Dec 10, 2020

I get the same issue (1.4.9 & 1.3.1). It does not show after logout.
I use it with ['failed_attempts'] = 0;
No whitelist

@arafatx
Copy link

arafatx commented Dec 26, 2020

It seems like you are using unsecured connection. Perhaps this might be the issue. Try using https and see if problem persists

@smefa
Copy link

smefa commented Dec 27, 2020

My site is using https with a valid certificate and I get the same issue. Contact me if you want to check it.

@claudiosoprano
Copy link
Author

claudiosoprano commented Dec 30, 2020 via email

@claudiosoprano
Copy link
Author

any news about this problem, it is still valid

@rorian
Copy link

rorian commented Jun 26, 2021

It's a great plugin but same problem here. After logout no recaptcha badge and no recaptacha callback possible. User needs to refresh the page.

-no other non-default plugins than rcguard

  • failed_attempts = 0
  • no whitelist
  • recaptcha v3

@dsoares
Copy link
Owner

dsoares commented Jun 26, 2021

I still can't reproduce the problem. Tested with latest roundcube version (master), PHP 7.4, the same options:

  • failed_attempts = 0
  • no whitelist
  • recaptcha v3
  • i suppose you're using the elastic skin

Is there any error in the browser console?

@rorian
Copy link

rorian commented Jun 26, 2021

I use elastic skin.
Some more details:

Rouncube 1.4.11

plugins:
archive | 3.4
filesystem_attachments | 1.0
jqueryui | 1.12.0
password | 5.1
rcguard | 1.3.1
zipdownload | 3.4

The steps are the same as @claudiosoprano screens.

  1. user logs into the panel using active recaptcha mechanism

  2. user clicks "Log out" which hrefs to /roundcube/?_task=logout from webmail panel

  3. after that the default the login screen shows up but this time with NO recaptcha badge.
    Further code insight gives that at this step, the <script src="https://www.google.com/recaptcha/api.js?render=keyhere></script> is missing in the code. No api fetch. Therfore no iframe also.
    No errors in console at all.

  4. at this point when user wants to log again he's getting CAPTCHA verification empty error message and..

  5. then page is doing an auto-refresh and -> the api.js is included, as before. reCaptcha shows - goto-step(1)

@pbiering
Copy link

pbiering commented Aug 1, 2023

can this be reproduced with latest release from here?

https://github.com/pbiering/roundcube-rcguard

If not, please close this issue.

@smefa
Copy link

smefa commented Aug 2, 2023

Looks ok to me now with Roundcube 1.6.2 & rcguard 1.3.2

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

6 participants