[Lift] Re: Bug in MappedEmail: emailPattern is wrong

2009-09-20 Thread Richard Dallaway

Apologies if I've duplicated the ticket, but I couldn't find it one,
and this issue is going to impact me too.

http://github.com/dpp/liftweb/issues/#issue/65

Cheers
Richard

On Fri, Sep 18, 2009 at 6:32 PM, David Pollak
feeder.of.the.be...@gmail.com wrote:
 Please open a ticket

 On Wed, Sep 16, 2009 at 11:27 AM, harryh har...@gmail.com wrote:

 Hrm, MappedEmail is now rejecting e-mails with a - in the part before
 the @.

 is %-+ being interpreted as % through + and not % OR - OR + ?

 -harryh, who is easily confused by regular expressions


--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
Lift group.
To post to this group, send email to liftweb@googlegroups.com
To unsubscribe from this group, send email to 
liftweb+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/liftweb?hl=en
-~--~~~~--~~--~--~---



[Lift] Re: Bug in MappedEmail: emailPattern is wrong

2009-09-18 Thread David Pollak
Please open a ticket

On Wed, Sep 16, 2009 at 11:27 AM, harryh har...@gmail.com wrote:


 Hrm, MappedEmail is now rejecting e-mails with a - in the part before
 the @.

 is %-+ being interpreted as % through + and not % OR - OR + ?

 -harryh, who is easily confused by regular expressions

 On Aug 26, 7:32 pm, David Pollak feeder.of.the.be...@gmail.com
 wrote:
  Thanks.
 
  Changed and pushed to GitHub.  Allow 2 hours for Maven availability.
 
  On Wed, Aug 26, 2009 at 4:15 PM, harryh har...@gmail.com wrote:
 
What's the change to the RegEx?
 
   val emailPattern = Pattern.compile(^[a-z0-9._%-+]+@(?:[a-z0-9-]+\\.)+
   [a-z]{2,4}$)
 
   note the addition of the + to the allowed characters before the @
 
   -harryh
 
  --
  Lift, the simply functional web frameworkhttp://liftweb.net
  Beginning Scalahttp://www.apress.com/book/view/1430219890
  Follow me:http://twitter.com/dpp
  Git some:http://github.com/dpp
 



-- 
Lift, the simply functional web framework http://liftweb.net
Beginning Scala http://www.apress.com/book/view/1430219890
Follow me: http://twitter.com/dpp
Git some: http://github.com/dpp

--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
Lift group.
To post to this group, send email to liftweb@googlegroups.com
To unsubscribe from this group, send email to 
liftweb+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/liftweb?hl=en
-~--~~~~--~~--~--~---



[Lift] Re: Bug in MappedEmail: emailPattern is wrong

2009-09-16 Thread harryh

Hrm, MappedEmail is now rejecting e-mails with a - in the part before
the @.

is %-+ being interpreted as % through + and not % OR - OR + ?

-harryh, who is easily confused by regular expressions

On Aug 26, 7:32 pm, David Pollak feeder.of.the.be...@gmail.com
wrote:
 Thanks.

 Changed and pushed to GitHub.  Allow 2 hours for Maven availability.

 On Wed, Aug 26, 2009 at 4:15 PM, harryh har...@gmail.com wrote:

   What's the change to the RegEx?

  val emailPattern = Pattern.compile(^[a-z0-9._%-+]+@(?:[a-z0-9-]+\\.)+
  [a-z]{2,4}$)

  note the addition of the + to the allowed characters before the @

  -harryh

 --
 Lift, the simply functional web frameworkhttp://liftweb.net
 Beginning Scalahttp://www.apress.com/book/view/1430219890
 Follow me:http://twitter.com/dpp
 Git some:http://github.com/dpp
--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
Lift group.
To post to this group, send email to liftweb@googlegroups.com
To unsubscribe from this group, send email to 
liftweb+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/liftweb?hl=en
-~--~~~~--~~--~--~---



[Lift] Re: Bug in MappedEmail: emailPattern is wrong

2009-08-27 Thread Viktor Klang
On Thu, Aug 27, 2009 at 10:27 AM, Marc Boschma
marc+lift...@boschma.cxmarc%2blift...@boschma.cx
 wrote:

 Personally I use:


 ^(?:[a-z0-9!#$%'*+/=?^_`{|}~-]+(?:\.[a-z0-9!#$%'*+/=?^_`{|}~-]+)*|(?:[\x01-\x08\x0b\x0c\x0e-\x1f\x21\x23-\x5b\x5d-\x7f]|\\[\x01-\x09\x0b\x0c\x0e-\x7f])*)@(?:[a-z0-9](?:[a-z0-9-]*[a-z0-9])?\.)+[a-z0-9](?:[a-z0-9-]*[a-z0-9])?$


Impressive!

It still doesn't tell you if the email is real or not ;)



 Marc

 On 27/08/2009, at 9:32 AM, David Pollak wrote:

 Thanks.

 Changed and pushed to GitHub.  Allow 2 hours for Maven availability.

 On Wed, Aug 26, 2009 at 4:15 PM, harryh har...@gmail.com wrote:


  What's the change to the RegEx?

 val emailPattern = Pattern.compile(^[a-z0-9._%-+]+@(?:[a-z0-9-]+\\.)+
 [a-z]{2,4}$)

 note the addition of the + to the allowed characters before the @

 -harryh




 --
 Lift, the simply functional web framework http://liftweb.net
 Beginning Scala http://www.apress.com/book/view/1430219890
 Follow me: http://twitter.com/dpp
 Git some: http://github.com/dpp





 



-- 
Viktor Klang

Blog: klangism.blogspot.com
Twttr: viktorklang

Lift Committer - liftweb.com
AKKA Committer - akkasource.org
Cassidy - github.com/viktorklang/Cassidy.git
SoftPub founder: http://groups.google.com/group/softpub

--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
Lift group.
To post to this group, send email to liftweb@googlegroups.com
To unsubscribe from this group, send email to 
liftweb+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/liftweb?hl=en
-~--~~~~--~~--~--~---



[Lift] Re: Bug in MappedEmail: emailPattern is wrong

2009-08-27 Thread Marc Boschma
Your right... I've been thinking about that and how the current mapper  
only does the regex check.. could always override the validate method...

Marc
On 27/08/2009, at 6:39 PM, Viktor Klang wrote:



 On Thu, Aug 27, 2009 at 10:27 AM, Marc Boschma marc+lift...@boschma.cx 
  wrote:
 Personally I use:

 ^(?:[a-z0-9!#$%'*+/=?^_`{|}~-]+(?:\.[a-z0-9!#$%'*+/=?^_`{|}~-] 
 +)*|(?:[\x01-\x08\x0b\x0c\x0e-\x1f\x21\x23-\x5b\x5d-\x7f]|\\[\x01- 
 \x09\x0b\x0c\x0e-\x7f])*)@(?:[a-z0-9](?:[a-z0-9-]*[a-z0-9])?\.)+[a- 
 z0-9](?:[a-z0-9-]*[a-z0-9])?$

 Impressive!

 It still doesn't tell you if the email is real or not ;)


 Marc

 On 27/08/2009, at 9:32 AM, David Pollak wrote:

 Thanks.

 Changed and pushed to GitHub.  Allow 2 hours for Maven availability.

 On Wed, Aug 26, 2009 at 4:15 PM, harryh har...@gmail.com wrote:

  What's the change to the RegEx?

 val emailPattern = Pattern.compile(^[a-z0-9._%-+]+@(?:[a-z0-9-]+\ 
 \.)+
 [a-z]{2,4}$)

 note the addition of the + to the allowed characters before the @

 -harryh




 -- 
 Lift, the simply functional web framework http://liftweb.net
 Beginning Scala http://www.apress.com/book/view/1430219890
 Follow me: http://twitter.com/dpp
 Git some: http://github.com/dpp









 -- 
 Viktor Klang

 Blog: klangism.blogspot.com
 Twttr: viktorklang

 Lift Committer - liftweb.com
 AKKA Committer - akkasource.org
 Cassidy - github.com/viktorklang/Cassidy.git
 SoftPub founder: http://groups.google.com/group/softpub

 


--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
Lift group.
To post to this group, send email to liftweb@googlegroups.com
To unsubscribe from this group, send email to 
liftweb+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/liftweb?hl=en
-~--~~~~--~~--~--~---



[Lift] Re: Bug in MappedEmail: emailPattern is wrong

2009-08-26 Thread harryh

 What's the change to the RegEx?

val emailPattern = Pattern.compile(^[a-z0-9._%-+]+@(?:[a-z0-9-]+\\.)+
[a-z]{2,4}$)

note the addition of the + to the allowed characters before the @

-harryh
--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
Lift group.
To post to this group, send email to liftweb@googlegroups.com
To unsubscribe from this group, send email to 
liftweb+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/liftweb?hl=en
-~--~~~~--~~--~--~---



[Lift] Re: Bug in MappedEmail: emailPattern is wrong

2009-08-26 Thread David Pollak
On Wed, Aug 26, 2009 at 3:15 PM, harryh har...@gmail.com wrote:


 It doesn't allow for + in the name part of the e-mail (harryh
 +l...@gmail.com).
 Can one of the lift commiters fix this?  It's a one character change.


What's the change to the RegEx?




 -harryh

 



-- 
Lift, the simply functional web framework http://liftweb.net
Beginning Scala http://www.apress.com/book/view/1430219890
Follow me: http://twitter.com/dpp
Git some: http://github.com/dpp

--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
Lift group.
To post to this group, send email to liftweb@googlegroups.com
To unsubscribe from this group, send email to 
liftweb+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/liftweb?hl=en
-~--~~~~--~~--~--~---



[Lift] Re: Bug in MappedEmail: emailPattern is wrong

2009-08-26 Thread David Pollak
Thanks.

Changed and pushed to GitHub.  Allow 2 hours for Maven availability.

On Wed, Aug 26, 2009 at 4:15 PM, harryh har...@gmail.com wrote:


  What's the change to the RegEx?

 val emailPattern = Pattern.compile(^[a-z0-9._%-+]+@(?:[a-z0-9-]+\\.)+
 [a-z]{2,4}$)

 note the addition of the + to the allowed characters before the @

 -harryh
 



-- 
Lift, the simply functional web framework http://liftweb.net
Beginning Scala http://www.apress.com/book/view/1430219890
Follow me: http://twitter.com/dpp
Git some: http://github.com/dpp

--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
Lift group.
To post to this group, send email to liftweb@googlegroups.com
To unsubscribe from this group, send email to 
liftweb+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/liftweb?hl=en
-~--~~~~--~~--~--~---