[PATCH v2] auth: let users log in using their email address

Mads Kiilerich mads at kiilerich.com
Mon May 25 16:57:10 EDT 2015


On 05/17/2015 01:04 PM, Andrew Shadura wrote:
>
> Hello,
>
> On Sun, 17 May 2015 03:12:20 +0200
> Mads Kiilerich <mads-h1iwDtBV6gO1Z/+hSey0Gg at public.gmane.org> wrote:
>
>>> Correct me if I'm wrong, but if I read the code correctly, the check
>>> here will have no effect (which is why I haven't added it).
>> Ok. The explanation explains it. The code do however seem fragile and
>> non-obvious when reading it. An extra check or a clear comment would
>> help.
> So adding a comment — and you're fine with the change? :)

Not really. I think we pin-pointed that it takes some existing technical 
debt and makes it worse ... and the feature still only works for some 
kind of logins and is thus not very usable.

/Mads



More information about the kallithea-general mailing list