I remember I looked at this issue at PBF but could not replicate it for some reasons. If we want to trim space like this, I think we should add filter="TRIM" to the field, before validate="email" https://github.com/joomla/joomla-cms/blob/4.4-dev/components/com_users/forms/reset_request.xml#L10 instead

Personaly this is expected and intended behaviour and should not be accepted

would move it to 5.2, btw. if you already trim the email you could lowercase it too in php but I think it doesn't has an performance impact in sql

that a great improvement for User Experience, thank you. I tested it but it does not work

I have tested this item ? unsuccessfully on 95be583

Firefox doesn't see it as an email address when I type in a space first.
Google Chrome does, and before AND after applying the patch, I get a message that the email is sent.

I have tested this item ✅ successfully on 95be583

Tested successfully with 5.2-dev

• The user used could not have the superuser role, as no password reset mails are sent to them
• It was already working before the patch as ¿all? modern browsers remove leading spaces for email fields (verified with Chrome developer tools inspecting network request)
• Used one and multiple spaces before email address
• Tested with 'Forgot your password?' and 'Forgot your username?'
• Using emails with spaces before is still working after applying the PR
• The interesting question is the used browser in the issue?

  ---

  _{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42893.}

@crommie I can confirm that Firefox is not accepting email address with leading space, but there is a clear error message Please enter an email address. In my view, this is valid browser behaviour and cannot and need not be influenced by this PR.

Email was sent before and after applying patch - in theory, because I tested on a PBF server site that didn't send emails at all. So maybe the unsuccessful test should be removed.

So maybe the unsuccessful test should be removed

mark "not tested" at issue trackerand "submit test result".

I have not tested this item.

Yayyy, that worked, thanks.

This pull request has been automatically rebased to 5.3-dev.

I have tested this item ✅ successfully on 95be583

I have tested this item successfully and it is working as expected

I have tested this item 🔴 unsuccessfully on 95be583

Using Firefox, password reset fails in any case, with or without patch. Firefox displays a little message «Please enter an email address» in either case. Maybe this issue is related to a certain brwoser?

With Chrome the problem cannot be reproduced, email is sent - no issue. Patch not needed or already fixed?

I have tested this item 🔴 unsuccessfully on 95be583

@MacJoom Can you please provide the reasons for the test result?

Without the patch the issue occurs in firefox, in chrome the password reset email is being sent, in firefox I see the message mentioned above. applying the patch doesn't change anything

@MacJoom Can you please provide the reasons for the test result?

See above

@MacJoom Thanks for the answer. I wasn't sure because of "Patch not needed or already fixed?".

I have tested this item 🔴 unsuccessfully on 95be583

Errormessage: The email address you entered is invalid.

I tested this in Chrome - entering an email with a space at the beginning worked with and without the patch. As for if this is needed or not, I am all for it since I cannot think of any circumstance where a person would intentionally put a space at the beginning of entering an email, so why not just save the frustration of it failing. Also, why not set things to remove a blank space at the end of an email as well, since sometimes double-clicking to select an email for cut and paste might accidentally grab a space that may be after it. Of course it is not needed if a space at the end doesn't break anything.

I have tested this item 🔴 unsuccessfully on 95be583

Sorry for the double post. I missed using the "Submit Test Result" feature - #noob!

I tested this in Chrome - entering an email with a space at the beginning worked with and without the patch. As for if this is needed or not, I am all for it since I cannot think of any circumstance where a person would intentionally put a space at the beginning of entering an email, so why not just save the frustration of it failing. Also, why not set things to remove a blank space at the end of an email as well, since sometimes double-clicking to select an email for cut and paste might accidentally grab a space that may be after it. Of course it is not needed if a space at the end doesn't break anything.

This pull request has been automatically rebased to 6.0-dev.

I have tested this item 🔴 unsuccessfully on ae0c40c

In Chrome browser the "Forgot your password?" works both with space before e-mail and space after - the request is send.
In Firefox browser the request is blocked by the browser if adding a space in fornt or after the e-mail.

This pull request has been automatically rebased to 6.1-dev.

I have tested this item ✅ successfully on c7650ba

This works in the current versions of Chrome and Safari with and without the patch, as these browsers allow spaces.

In Firefox, it does not work with or without the patch. However, this is due to the behaviour of the browser, and a message appears.

Screenshot: Firefox 146.0.1 (aarch64)

I have tested this item 🔴 unsuccessfully on c7650ba

It works with Edge but it does not work with Firefox.

I have tested this item 🔴 unsuccessfully. Using Firefox 147.0.2.
When using an e-mail address with space and pressing submit on /index.php/component/users/reset i get a message to fill in a correct e-mail address.

I have tested this item ✅ successfully. Using Chrome 144.0.7559.110