1549: The email-address 'myname@localhost' is not considered valid.
- WontFix
- Review Board
holmstro********@gmai***** (Google Code) (Is this you? Claim this profile.) | |
June 10, 2010 |
What version are you running? 1.0.5.1 What's the URL of the page containing the problem? /account/preferences/ What steps will reproduce the problem? 1. Login for the first time on a fresh installation 2. enter 'myname@localhost' as email address 3. submit What is the expected output? What do you see instead? I expect my account settings to be updated, instead I get the message 'Enter a valid e-mail address.' What operating system are you using? What browser? Linux vurt 2.6.31-20-generic #57-Ubuntu SMP Mon Feb 8 09:05:19 UTC 2010 i686 GNU/Linux Firefox/3.5.8 Python 2.6.4 Please provide any additional information below. Since I entered this email address when running rb-site it was accepted then this is not only a validation problem but also inconsistent behaviour. Django version installed is Django-1.1.1.
This is due to Django's validation. I don't think it's unreasonable to mark that as an invalid e-mail address, but we should certainly be consistent. Do you have a strong need for a non-FLQN in the e-mail address?
-
+ NeedInfo -
- Priority-Medium + Priority-Low + Component-Settings + Component-RB-Site + Milestone-Release1.5
In general I can sort of live with flawed email-address validation since this is so common and also understand djangos problem with altering their email-address validator. That said I think that not allowing localhost is kind of a nuisance when evaluating a product like reviewboard. During evaluation it would be nice to be able to simply use localhost to see how email is handled without spamming already existing email-accounts or setting up a fake host. This, of course, was not a big problem and you are certainly not the only ones that have it. I did however find that the inconsistency between the setup-tool(nice work by the way, very easy to use) and the application made me feel less confident about the overall quality of reviewboard, something which could easily be avoided.
Thanks. That's good feedback. We should make a decision as to how much validation we want to do, and where. Are there other things in the installer that you found to be inconsistent?
I didn't have any other huge problems with the installer but there was another thing you'd might consider. Since I was evaluating I sandboxed reviewboard and it's dependencies by using virtualenv, which made it very hard for me to use my systems apache or lighttpd. This was something I didn't think about until the setup was done but since lighttpd is easy to run standalone on any port this wasn't much of a problem, I just had to add server.document-root (I used /tmp/) and server.port to the generated lighttpd.conf to make it work(don't know if this will cover all cases). With all the problems concerning distribution packages and python packages I don't think I'm the only one that does this and it would've been nice to have some pointers in the documentation, output from rb-site or/and commented options in lighttpd.conf to make testing even simpler.