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

It's possible to create a account with wrong username capitalisation #594

Closed
JSoko opened this issue Jan 28, 2017 · 4 comments
Closed

It's possible to create a account with wrong username capitalisation #594

JSoko opened this issue Jan 28, 2017 · 4 comments
Assignees
Labels

Comments

@JSoko
Copy link
Member

JSoko commented Jan 28, 2017

see nextcloud/server#3305

Actual behaviour

If you have a user testuser you con connect with NextcloudAndroid entering the username Testuser
While uploading files you get a warning

Expected behaviour

No problems or don't connect because of the wrong capitalisation (T instead of t)

Steps to reproduce

see nextcloud/server#3305

@JSoko JSoko changed the title Its possible to create a account with wron username capitalisation It's possible to create a account with wrong username capitalisation Jan 28, 2017
@tobiasKaminsky
Copy link
Member

cc @LukasReschke what is the desired behaviour?

@mario
Copy link
Contributor

mario commented Feb 8, 2017

Seems to be a server issue - so closing it here.

@mario mario closed this as completed Feb 8, 2017
@nickvergessen
Copy link
Member

No the server can not handle this. You can get the correct user id casing from:
The cloud/user endpoint gives you the correct spelling in the <id> element:

curl -H "OCS-APIREQUEST: true" https://Admin:admin@localhost/ocs/v2.php/cloud/user
<?xml version="1.0"?>
<ocs>
 <meta>
  <status>ok</status>
  <statuscode>200</statuscode>
  <message>OK</message>
 </meta>
 <data>
  <id>admin</id>
  <display-name>Administrator</display-name>
 </data>
</ocs>

however not sure if that works with ldap, so I'd just use that if lower(id) === lower(provided id)

@nickvergessen nickvergessen reopened this Feb 9, 2017
@tobiasKaminsky
Copy link
Member

I tried it with my current NC and current android app and used "Tobi" instead of "tobi" and it is working without any issue. I tried both auto upload and manually upload.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants