dharamgollapudi / jaikuengine

Automatically exported from code.google.com/p/jaikuengine
Apache License 2.0
0 stars 0 forks source link

user not found errors when changing backgrounds #73

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
as reported here:

http://www.jaiku.com/channel/jaiku/presence/6f1e59cf69bc4f7282655cab6e5f165d
http://www.jaiku.com/channel/jaiku/presence/19fc989264c74dc4bf5c42330fab5a85

It appears to be the same case-sensitivity bug as before :/ 

Original issue reported on code.google.com by andyster on 31 Mar 2009 at 5:35

GoogleCodeExporter commented 9 years ago
I've created a user with mixed case username but I still can't manage to 
reproduce this running locally. Anyone 
else?

Original comment by jonasnoc...@gmail.com on 31 Mar 2009 at 9:15

GoogleCodeExporter commented 9 years ago
I don't believe we'll ever be able to reproduce this problem. It only happens to
people who have accounts on jaiku.com that have been migrated. My guess is that
something went wrong and now the normalized_nick for these users is not 
lowercase.

For example look at this post:
http://www.jaiku.com/channel/jaiku/presence/a4e5edd6da134cc9b1a1c371da825dc1 
Note
that the user does exist and has a url with a lowercase domain name which works.
Here's what seems to be happening: In various places in the codebase (to do with
changing backgrounds, changing passwords and sending invites) we're using the
unnormalised/mixed-case nickname of the user and this is failing to find a 
match.
However when we report the error we use normalised/lower-case name of the user.

Andy: Can you use the AppEngine data-viewer and examine the Actor entity for the
affected users. Do they have lower-case normalized_nick values? It might also 
be a
good idea to dig up the migration code and find out if it did anything special 
when
migrating users with mixed-case usernames.

Original comment by adewale on 14 Apr 2009 at 12:13

GoogleCodeExporter commented 9 years ago
Looks like I was wrong. This new user: 
http://www.jaiku.com/channel/jaiku/presence/62553f0098954e75b70149d741ad244b is 
suffering from the 
same bug. He has a mixed-case username but his account didn't go through the 
data migration process.

Original comment by adewale on 24 Apr 2009 at 7:40

GoogleCodeExporter commented 9 years ago
Being reviewed at http://rietku.appspot.com/13001

Original comment by andyster on 27 Apr 2009 at 7:06

GoogleCodeExporter commented 9 years ago
fixed in r65

Original comment by andyster on 28 Apr 2009 at 1:45