Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

hostnames <table> in greeting banner does not consider ipv6 #1067

Open
pywy18 opened this issue Jun 2, 2020 · 4 comments
Open

hostnames <table> in greeting banner does not consider ipv6 #1067

pywy18 opened this issue Jun 2, 2020 · 4 comments
Assignees

Comments

@pywy18
Copy link

pywy18 commented Jun 2, 2020

Using opensmtpd on OpenBSD 6.7 stable here

In a configuration using a table for greeting banner, the ipv6 adresses are not considered :
listen on em0 port 25 hostnames <myhostnames>
with the table content :

192.168.0.1   my.custom.domain.tld
2001:db8:0::1  another.custom.domain.tld

Whenever I connect on the ipv4 addr, the right domain shows up, but the default domain is displayed when I connect using ipv6, in stead of the one specified in table.

@robert-scheck
Copy link

2001:db8:0::1

Is 2001:db8::1 leading to the same behaviour?

@poolpOrg poolpOrg self-assigned this Dec 2, 2020
@poolpOrg
Copy link
Member

poolpOrg commented Dec 2, 2020

Issue reproduced and understood, will work on a fix.

Basically, the problem is that for multi-column tables, the table parser in table_static.c uses the first ':' as a field separator since it is the delimiter for aliases. In this cases, this results in:

2001:db8:0::1  another.custom.domain.tld

being understood as:

2001 = db8:0::1  another.custom.domain.tld

@poolpOrg
Copy link
Member

Will let upstream know about this issue so they can fix

@innogrey
Copy link

Was this issue ever resolved? It seems that 6.8.0p2 still doesn't allow putting v6 addresses in a table.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants