Skip to content
This repository has been archived by the owner on Dec 8, 2021. It is now read-only.

Any password should be usable for the annotation service #87

Open
benel opened this issue Nov 14, 2019 · 4 comments
Open

Any password should be usable for the annotation service #87

benel opened this issue Nov 14, 2019 · 4 comments
Labels

Comments

@benel
Copy link
Member

benel commented Nov 14, 2019

Among 130 students, 2 of them were unable to login to Argos through LaSuli, whereas they succeeded in Cassandre.

Because authentication settings are exactly the same on Argos and Cassandre,
and because of the scarcity of the bug,
I suspect password encoding to be not robust enough in LaSuli (contrary to Cassandre).

@christophe-lejeune Do you remember a similar bug you fixed in Cassandre ?

@benel benel added the Fix label Nov 14, 2019
@christophe-lejeune
Copy link
Member

I have recently restored one old rule in Cassandre code, that forces usernames (not passwords, of course) to be lowercase (ie: usernames are converted lowercase, whatever the user fills in the username field). Passwords remain untouched.

The reason for this rule was that our software (Cassandre + AAAforREST + LDAP) allows users to register with their username, even if they submit it with a mix of lower and uppercase (this is the case, in particular, for people using phones or tablets). I realized later that one username may create two (or more) different identities (corresponding to each mix of lower and uppercase, for the same username). Perhaps you are facing a similar problem (?).

I have to mention that, last yeay, I also faced a problem with one and only user, reporting that he was unable to log in Cassandre. However, given this student was not sure of his own password and because he did not stop to change his password (in order to solve the problem), I decided to ignore the problem (assuming that it was not coming from the software iteself).

@benel
Copy link
Member Author

benel commented Nov 14, 2019

I see that Cassandre use serialize from jQuery which does URI encoding.

In contrast, LaSuli gets the value untouched.

@benel
Copy link
Member Author

benel commented Nov 14, 2019

I will add password URI encoding in the next release of LaSuli.
Thanks for your help :)

@benel benel closed this as completed in b5a3a8e Nov 18, 2019
@benel benel reopened this Dec 1, 2020
@benel
Copy link
Member Author

benel commented Dec 1, 2020

It seems that the problem still exists (maybe with + or &).
The issue needs further investigations.

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

No branches or pull requests

2 participants