FriendsOfFlarum / recaptcha

Increase your forum's security with Google reCAPTCHA
https://discuss.flarum.org/d/18399
MIT License
13 stars 11 forks source link

V 1.2.0 issue with ReCaptcha on login #39

Closed PeopleInside closed 1 year ago

PeopleInside commented 1 year ago

immagine

Bad visual on login

PeopleInside commented 1 year ago

Also from my test is impossibile to login if Captcha is active on Login

PeopleInside commented 1 year ago

The captcha is also showed wrong in the forget password and is also not allowing to get email for reset the password.

imorland commented 1 year ago

@PeopleInside I don't see any issues testing this locally or on my site.

Can you confirm that you have cleared the cache fully, along with your browser's cached js?

PeopleInside commented 1 year ago

yes I confirm. I'm reverting back to 1.1.1.

I installed the update, cleared cache and tested in incognito. Unable to login, unable to receive password reset and captcha showed wrongly in the new pages.

imorland commented 1 year ago

Please provide the output of php flarum info, as I cannot replicate the issues you describe here. From your screenshot, it looks like you have a custom theme? Have you tried disabling that for a moment to test?

PeopleInside commented 1 year ago

I use Asirem and yes I disabled it, still see the issue.

flarum info

Flarum core: 1.8.1
PHP version: 8.1.20
MySQL version: 10.6.12-MariaDB-0ubuntu0.22.04.1
Loaded extensions: Core, date, libxml, openssl, pcre, zlib, bz2, calendar, ctype, hash, filter, ftp, gettext, gmp, json, iconv, SPL, pcntl, readline, Reflection, session, standard, mbstring, SimpleXML, sockets, tokenizer, xml, mailparse, mysqlnd, bcmath, curl, dba, dom, enchant, fileinfo, gd, imagick, imap, intl, ldap, exif, mysqli, odbc, PDO, pdo_mysql, PDO_ODBC, pdo_pgsql, pdo_sqlite, pgsql, Phar, posix, pspell, redis, soap, sodium, sqlite3, sysvmsg, sysvsem, sysvshm, tidy, xmlreader, xmlwriter, xsl, zip, Zend OPcache
+-------------------------------------+---------------+--------+
| Flarum Extensions                   |               |        |
+-------------------------------------+---------------+--------+
| ID                                  | Version       | Commit |
+-------------------------------------+---------------+--------+
| flarum-flags                        | v1.8.0        |        |
| flarum-suspend                      | v1.8.0        |        |
| flarum-approval                     | v1.8.0        |        |
| flarum-tags                         | v1.8.0        |        |
| flarum-sticky                       | v1.8.0        |        |
| flarum-lock                         | v1.8.0        |        |
| afrux-forum-widgets-core            | v0.1.7        |        |
| fof-spamblock                       | 1.1.0         |        |
| flarum-subscriptions                | v1.8.0        |        |
| fof-byobu                           | 1.3.1         |        |
| zerosonesfun-composer-preview       | 1.2           |        |
| v17development-seo                  | v1.8.0        |        |
| sycho-profile-cover                 | v1.3.5        |        |
| noriods-auto-more                   | v1.0.0        |        |
| nearata-twofactor                   | v2.2.1        |        |
| kilowhat-audit-free                 | 1.6.0         |        |
| justoverclock-welcomebox            | 2.0.2         |        |
| ianm-synopsis                       | 1.3.0         |        |
| fof-username-request                | 1.1.2         |        |
| fof-user-bio                        | 1.2.0         |        |
| fof-terms                           | 1.2.1         |        |
| fof-subscribed                      | 1.1.3         |        |
| fof-stopforumspam                   | 1.2.3         |        |
| fof-split                           | 1.1.0         |        |
| fof-share-social                    | 1.1.2         |        |
| fof-recaptcha                       | 1.1.1         |        |
| fof-nightmode                       | 1.5.2         |        |
| fof-moderator-notes                 | 1.2.0         |        |
| fof-merge-discussions               | 1.4.0         |        |
| fof-links                           | 1.1.4         |        |
| fof-linguist                        | 1.1.1         |        |
| fof-ignore-users                    | 1.2.0         |        |
| fof-formatting                      | 1.0.3         |        |
| fof-follow-tags                     | 1.1.7         |        |
| fof-drafts                          | 1.2.2         |        |
| fof-disposable-emails               | 1.0.0         |        |
| fof-default-user-preferences        | 1.2.0         |        |
| fof-cookie-consent                  | 1.1.1         |        |
| flarum-statistics                   | v1.8.0        |        |
| flarum-pusher                       | v1.8.0        |        |
| flarum-package-manager              | v1.0.0-beta.2 |        |
| flarum-mentions                     | v1.8.0        |        |
| flarum-markdown                     | v1.8.0        |        |
| flarum-likes                        | v1.8.0        |        |
| flarum-lang-italian                 | 1.11.0        |        |
| flarum-lang-english                 | v1.8.0        |        |
| flarum-emoji                        | v1.8.0        |        |
| flarum-bbcode                       | v1.8.0        |        |
| flarum-akismet                      | v1.8.0        |        |
| dem13n-topic-starter-label          | 0.1.8         |        |
| clarkwinkelmann-first-post-approval | 1.0.1         |        |
| clarkwinkelmann-emojionearea        | 1.0.0         |        |
| blomstra-user-filter                | 0.1.4         |        |
| afrux-online-users-widget           | v0.1.6        |        |
| afrux-news-widget                   | v0.1.1        |        |
| afrux-forum-stats-widget            | v0.1.1        |        |
| afrux-asirem                        | v0.1.8        |        |
+-------------------------------------+---------------+--------+
PeopleInside commented 1 year ago

I attach two other screen with the error (no message) on login and reset password where the message error say that g-recaptcha response is required.

For now I disabled the re-captcha from reset and login page as is not working. Invisible re captcha.

I hope an update will resolve the issues in the two pages (reset and login).

Immagine 2023-06-28 134500 Immagine 2023-06-28 134707

dsevillamartin commented 1 year ago

I can reproduce this, and know what is causing the issue. I'll release a fix for it soon.

PeopleInside commented 1 year ago

Version 1.2.1 hide completely the ReCaptcha this is not good. The ReCaptcha is no more visible, a small logo with cookie and terms links should be showed

immagine

Update: all work, was an option on an extension ublock that hide the logo, thanks for the update. Good update