De-sensitize case for e-mail for signing in

Ticket

De-sensitize case for e-mail for signing in+status
De-sensitize case for e-mail for signing in+priority
De-sensitize case for e-mail for signing in+tag
De-sensitize case for e-mail for signing in+commit
 

 

De-sensitize case for e-mail for signing in+solution

Migrate existing stored emails to lower case.

 

Convert registering email addresses to lower case.

 

Convert signing in email address to lower case before testing.

 

 

I did these three, and in the process I made a couple of other changes:

  • I added a scope (set to :login) to validates_uniqueness, so that users with "logins" (which are really more like "codenames" now) can have the same email as an actual user.  So, for example, I could create a new wagn and have my Wagn Bot use , while still using that email for my personal account.
  • I had the migration get rid of logins for all the accounts that used to have them
  • I also had the migration set status to "system" on wagn bot and anon cards, so that they don't require passwords.  This gets them in line with the way we're installing them now.

you know, I could've migrated all account requests to have the "pending" status, but I'm guessing this one will sort itself out.

  --Ethan McCutchen.....Sun Nov 15 16:38:39 -0800 2009


migrations have been run on ang. fwiw, migrations there seem to be breaking on several dwagns:

 

osoeco, orchard, new2, earlysteps, grouppatterns. not vital that we fix these, but we should certainly watch closely when we next migrate pinz.

  --Ethan McCutchen.....Sun Nov 15 16:40:57 -0800 2009


http://www.dwagn.org/wagn/George_Por+*email has caps in it.

  --John Abbe.....Wed Dec 02 14:40:42 -0800 2009


Also still with caps:

http://www.dwagn.org/wagn/Jillian_Ada_Burrows+*email

http://www.dwagn.org/wagn/Bobbi_Newman+*email

http://www.dwagn.org/wagn/Rory+*email

http://www.dwagn.org/wagn/Winston+*email

http://www.dwagn.org/wagn/TED_HAND+*email (and unlinked, like Markus Strauss etc. below)

http://www.dwagn.org/wagn/Paul_B_Hartzog+*email

http://www.dwagn.org/wagn/Kenneth_MacClune+*email

 

Has an email address, but it doesn't auto-link, which seems odd:

http://www.dwagn.org/wagn/Markus_Strauss+*email

 

And with no email at all:

http://www.dwagn.org/wagn/Johnny_Cake+*email

http://www.dwagn.org/wagn/TestUser+*email

http://www.dwagn.org/wagn/Bluser+*email

http://www.dwagn.org/wagn/Wagn_Bot+*email (intentional?)

  --John Abbe.....Wed Dec 02 14:47:58 -0800 2009


Everything else seems good.

 

We'll have to make sure to take care of this when implementing mass-import of account info. (didn't see a ticket for this in http://www.wagn.org/wagn/account+tickets_for_item but i know it's something we talked about)

  --John Abbe.....Wed Dec 02 15:04:18 -0800 2009


we made a new migration to fix the buggeredness. migrating now.

 

not too worried about mass-import, because I have it autodowncasing as part of validation.

  --Ethan McCutchen.....Thu Dec 03 10:12:58 -0800 2009


Has an email address, but it doesn't auto-link, which seems odd:

http://www.dwagn.org/wagn/Markus_Strauss+*email

 

http://www.dwagn.org/wagn/TED_HAND+*email (this one wasn't becoming a link, but now is)

 

No email at all:

http://www.dwagn.org/wagn/Johnny_Cake+*email

http://www.dwagn.org/wagn/TestUser+*email

http://www.dwagn.org/wagn/Bluser+*email

http://www.dwagn.org/wagn/Wagn_Bot+*email (intentional?)

  --John Abbe.....Thu Dec 03 15:36:09 -0800 2009


1. added eu to accepted domain endings. tested markus' email locally and it worked.

2. Guessing Ted's didn't work because .net was capitalized or something?

3. Wagn Bot is a system account and therefore doesn't have to have an email. Can if you want...

4. would be good to add emails to those other accounts on wagn.org. can be , bogus2... they'll get copied over to dwagn automatically eventually. we may want to do some data checking on other sites later on. Idea?

  --Ethan McCutchen.....Fri Dec 04 08:39:51 -0800 2009


1. Any reason not to allow all top level domains? (i can probably dig up a canonical list if that would be helpful; this would become a periodic task to check and update the list as they add domains)

3. Vague memory that some emails from a Wagn come from Wagn Bot's address if no other address is specified. If this is the case, then we probably do want something there.

4. (and 3.) Not seeing/thinking how to add addresses (or maybe were you suggesting it be an Idea because it requires console access?)

  --John Abbe.....Fri Dec 04 10:08:33 -0800 2009


1. it's not about disallowing accounts, it's about pattern matching within text. there's a list of domain endings that keeps us from matching things that aren't domains. +*email isn't special, it's just treated as normal text.

2. yeah, it uses wagbot if you don't have *account+*from or *notify+*from. Our sites should probably have those set. As for english and other sites, you can put something bogus in there if you want to, but please don't use any of our domains. I don't want to have a bunch of crap pinging from other folks installations.

4. um, you add email addresses to a card with an account by clicking on the options tab on that card. if the card doesn't have an account, then you won't see the interface. Looks to me like that's the issue with at least some those cards: No account. Not all user cards have accounts. so let's forget about them.

  --Ethan McCutchen.....Fri Dec 04 10:34:48 -0800 2009


1. Bad choice of words. I meant, is there any reason not to match all ICANN-approved top level domains? Important for i18n among other things. Shall i create a ticket?

2. Pondering; don't really want to spam anyone else either.

  --John Abbe.....Fri Dec 04 12:56:48 -0800 2009