Re: smtpd: fix ipv6 table lookups [was: Re: IPv6 and addrname SNAFU]

2023-12-15 Thread Kirill Miazine
• Kirill Miazine [2023-12-15 10:11]: In my previous mail I failed to realize that the colon is needed for aliases(5) so we can't "just" get rid of it.  (thanks Kirill for making me noticing it -- I completely forgot that file tables split on : too.) I don't have any other ideas.  Maybe we can

Re: smtpd: fix ipv6 table lookups [was: Re: IPv6 and addrname SNAFU]

2023-12-15 Thread Kirill Miazine
• Omar Polo [2023-12-15 10:04]: Hi, On 2023/12/14 23:15:53 +, gil...@poolp.org wrote: hello, fyi, if the first line of a file used to feed a table is @list, then it forces parsing to consider content as a serie of values, not a serie of key-value. Yeah, the @list special comment is ha

Re: smtpd: fix ipv6 table lookups [was: Re: IPv6 and addrname SNAFU]

2023-12-15 Thread Omar Polo
Hi, On 2023/12/14 23:15:53 +, gil...@poolp.org wrote: > hello, > > fyi, > if the first line of a file used to feed a table is @list, then > it forces parsing to consider content as a serie of values, not > a serie of key-value. Yeah, the @list special comment is handy to avoid ambiguities be

Re: smtpd: fix ipv6 table lookups [was: Re: IPv6 and addrname SNAFU]

2023-12-14 Thread Kirill Miazine
• Omar Polo [2023-12-14 23:33]: > On 2023/12/14 21:36:44 +0100, Kirill Miazine wrote: > > [...] > > cvs checkout took looong time, so I took src.tar.gz from 7.4, applied > > patch there. > > > > diff worked for static maps, at least. tested on this config: > > thanks for testing the diff > > >

Re: smtpd: fix ipv6 table lookups [was: Re: IPv6 and addrname SNAFU]

2023-12-14 Thread gilles
hello, fyi, if the first line of a file used to feed a table is @list, then it forces parsing to consider content as a serie of values, not a serie of key-value. gilles December 14, 2023 11:33 PM, "Omar Polo" wrote: > On 2023/12/14 21:36:44 +0100, Kirill Miazine wrote: > >> [...] >> cvs ch

Re: smtpd: fix ipv6 table lookups [was: Re: IPv6 and addrname SNAFU]

2023-12-14 Thread Omar Polo
On 2023/12/14 21:36:44 +0100, Kirill Miazine wrote: > [...] > cvs checkout took looong time, so I took src.tar.gz from 7.4, applied > patch there. > > diff worked for static maps, at least. tested on this config: thanks for testing the diff > [...] > > but didn't work with "table helo-names f

Re: smtpd: fix ipv6 table lookups [was: Re: IPv6 and addrname SNAFU]

2023-12-14 Thread Kirill Miazine
Helo, Omar Thank you for the patch! • Omar Polo [2023-12-14 20:01]: [moving to tech@] On 2023/12/13 20:37:09 +0100, Kirill Miazine wrote: I've spent several hours debugging an issue. table(5) specifies addrname format as a mapping from inet4 or inet6 addresses to hostnames: ::

smtpd: fix ipv6 table lookups [was: Re: IPv6 and addrname SNAFU]

2023-12-14 Thread Omar Polo
[moving to tech@] On 2023/12/13 20:37:09 +0100, Kirill Miazine wrote: > I've spent several hours debugging an issue. > > table(5) specifies addrname format as a mapping from inet4 or inet6 > addresses to hostnames: > > ::1 localhost > 127.0.0.1 localho