Skip to content

LDAP integration is broken in current edge #3692

@reneploetz

Description

@reneploetz

📝 Preliminary Checks

  • I tried searching for an existing issue and followed the debugging docs advice, but still need assistance.

👀 What Happened?

This PR changed the virtual_mailbox_maps to texthash:/etc/postfix/vmailbox. This is only useful for file-account-based systems.

The LDAP integration needs postconf 'virtual_mailbox_maps = ldap:/etc/postfix/ldap-users.cf' but is configured before that, so the correct setting is overwritten by the new code.
This leads postfix being unable process incoming messages.

We need to tweak this line to check for LDAP too or use a different order.

For anyone affected using edge:
Use a user-patches.sh with:
postconf 'virtual_mailbox_maps = ldap:/etc/postfix/ldap-users.cf'

👟 Reproduction Steps

Enable LDAP support via ACCOUNT_PROVISIONER=LDAP.

🐋 DMS Version

edge

💻 Operating System and Architecture

Debian 11

⚙️ Container configuration files

No response

📜 Relevant log output

Dec 10 17:16:42 mail postfix/smtpd[1247]: warning: texthash:/etc/postfix/vmailbox is unavailable. open database /etc/postfix/vmailbox: No such file or directo
ry                                                         
Dec 10 17:16:42 mail postfix/smtpd[1247]: warning: texthash:/etc/postfix/vmailbox lookup error for "[email protected]"
Dec 10 17:16:42 mail postfix/smtpd[1247]: NOQUEUE: reject: RCPT from localhost[::1]: 451 4.3.0 <r
[email protected]>: Temporary lookup failure; from=<[email protected]> to=<[email protected]> proto=ESMTP helo=<mail.example.com>

Metadata

Metadata

Assignees

No one assigned

    Labels

    kind/bug/reportA report about a bugmeta/needs triageThis issue / PR needs checks and verification from maintainers

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions