Opened 12 years ago

Last modified 12 years ago

#366 new defect

@local_domains_maps doesn't work as expected, not needed?

Reported by: dmorton Owned by: dmorton
Priority: normal Milestone:
Component: General Version: 1.0.1
Severity: normal Keywords:
Cc:

Description

The amavisd.conf file supplied has this line:

@local_domains_maps = ( [".$mydomain"] );

But it doesn't work as expected for me... I turned on autocreate, sent a message to a new account. I had removed the domain default for @dgrmm.net, but had dgrmm.net in the $mydomain variable, which I would expect to call it a local address.

In all fairness, I think this config item should not be present; the definitive answer for whether an address is local should be if we have a database entry other than @. to handle it.

Besides removing this from the config file, there is a bunch of code in maia_autocreate_user() that can be removed, as it would be impossible to autocreate a nonlocal(@.) user.

Change History (2)

comment:1 Changed 12 years ago by rjl

There are a number of obsolete amavisd-new config items like that one, yes. Most of the *_maps() settings aren't needed in a Maia setup since those values are typically stored in SQL columns and rows. The thing to remember about amavisd-new is that it was designed to be very flexible--there are generally three or four different ways to achieve the same configuration, with or without SQL, with or without LDAP, etc. The *_maps() settings are a convenience for barebones (non-SQL) setups that need to hard-code user configs in the amavisd.conf file.

Maia users can get into trouble if they set some of these things in amavisd.conf as well as in SQL--that's a downfall of the amavisd-new flexibility philosophy: if there are three different ways to configure a given setting, there are nine potential ways to misconfigure that setting. If we wanted to get really draconian about it, we could rip out almost all the non-SQL configuration options from amavisd-maia, but it would have to be some very delicate surgery.

comment:2 Changed 12 years ago by dmorton

  • Milestone 1.0.2 deleted

commented out the line in amavisd.conf in [1111], but leaving ticket open to investigate the latter half of the ticket. Removing from 1.0.2 milestone.

Note: See TracTickets for help on using tickets.