Re: [PATCH v4 1/3] send-mail: improve checks for valid_fqdn

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> Yuck, can we do something about this overly long mess?  
> 
> If use a temporary $subdomain = '(?!-)[A-Za-z0-9-]{1,63}(?<!-)',
> would it make it easier to spot the repetition in the structure,
> i.e. /^$subdomain(?:\.$subdomain)*$/ and make it less error prone
> when somebody has to touch lines around here, or something?
> 
> But other than that, looking good.
> 
> Even though this allows "foo.local", on "darwin" it is still not
> allowed, which sounds like a regression-free change

Sent a version 5 here: https://lore.kernel.org/git/PN3PR01MB95974932FF37D9F24A7633C6B88BA@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx/T/#t

I forgot to add --in-reply-to while using git send-email to send v5, so it got sent
as a separate thread by mistake.




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux