'tis the season to get annoyed

Dec 16, 2008 15:26

Et tu, ThinkGeek? Like most of the sites I register for that actually support it, I registered for this one using an email address with a plus-sign in it[1]. Today, attempting to do a little Christmas Shopping, I noticed that the email field on the shipping address was empty, and filled in the exact same address as I'd registered with. And the site ( Read more... )

thinkgeek, geek, customer abuse

Leave a comment

Comments 5

anavolena December 16 2008, 17:22:25 UTC
It never ceases to amaze me the frequency with which people write Yet Another Email Or Phone Validation routine when plenty of established, tested OSS packages out there to use.

Reply

waider December 16 2008, 20:10:54 UTC
See comment below from the ThinkGeek monkeys (publicity fixes bugs faster than bug reports...)

Reply


ThinkGeek monkey update anonymous December 16 2008, 18:02:18 UTC
Greetings. I come bearing good checkout news. We caught your issue after @bosca fed this post into the Twitterverse this morning, and have just taken care of the plus sign problem.

The backstory: We updated our checkout process a few weeks ago with some new AJAX code that happened to be a bit buggy. It is now bug-free.

Thanks for the catch,

ThinkGeek Monkeys
@thinkgeek

Reply


(The comment has been removed)

waider December 16 2008, 20:09:59 UTC
Accepting the plus sign as a valid email character is. Having a RFC (2)822 parser which can cope with it, written in Javascript or the back-end language of your choice, is an entirely separate issue. It's not the ThinkGeek mailer that doesn't cope, it's their front-end (see comment above from the monkeys themselves).

Reply

(The comment has been removed)

waider December 16 2008, 20:33:40 UTC
To be honest, what annoys me most about the plus-is-space problem is that it's in the same bucket, more or less, as the spaces-vs.-tabs problem in Makefiles: it could have been fixed a long time ago by dropping support for the plainly broken behaviour, but (and I am assuming here) this was not done to avoid breaking the user experience for the miniscule userbase that existed at the time. Stupid, shortsighted, and stupid once again.

Reply


Leave a comment

Up