Upload a Joomla! package in com_joomlaupdate. When it asks for credential, enter correct credentials but use different case for username (e.g. if your username is admin, enter Admin)
Confirmation successful, installation proceeds.
403 Access Forbidden error.
Joomla! 3.6.4, PHP 7.0.9.
I'm not sure if this is intentional or not but it is inconsistent with other login forms. If this is intentional, case-sensitive requirement should at least be indicated in the form.
Labels |
Added:
?
|
EDIT:
Sorry, this concerns passwords, not User name
lol - you had me confused there
I am still extremely confused: why do we accept lower and upper entries for the username in the other login forms (just tested and indeed, one can login).
This looks wrong to me.
fairly common for usernames not to be case sensitive.
But SharkyKz is right on the updater (captive login) we are case sensitive for usernames on the login not.
@infograf768 yep that's true. But in the first case the behavior should be equal on all areas.
@zero-24 Any reason you don't use strcasecmp
?? :-)
Folks, I just tested a new install and I confirm that sql does NOT accept case-insensitive Username.
Why would Joomla accept?
Not just Joomla - any system that uses an email address as the username for example will be case insensitive
Category | ⇒ | com_joomlaupdate |
Status | New | ⇒ | Confirmed |
I'm with Brian it is extremely common from a user perspective to allow upper and lower case in username which can be used for login. Many times the user may start with a capital for example if writing on an iPhone and so on. It's accepted because users can be stupid (I know way too much after seeing this first hand after laughing a website for a client recently and they didn't even read the bright login they click to continue before logging in).
I would use @zero-24 solution too.
It's accepted because users can be stupid
The reason is more that the username isn't considered a secret element. It's also not saved encrypted in the database and sometimes even is shown on the page. Thus case sensitivity isn't of concern.
Only the password is considered a secret and thus it has to match exactly.
Either way it's a major inconsistency. This is the only part of the system that seems to have a case sensitive requirement so should be adjusted (it looks like Tobias' patch would do the trick).
@zero-24 can you please make a Pr based on staging...zero-24:username_login so that we can close this and move on
Status | Confirmed | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-05-21 14:09:02 |
Closed_By | ⇒ | zero-24 |
Done. looks like i have forgotten this over the time. Thanks for the reminder.
I'm not 100% sure but please try this: staging...zero-24:username_login I can make a PR if that works. Thanks!