🪲: Default DKIM signing logic does not work

[provokateurin]

What happened?

stalwart/crates/common/src/config/smtp/auth.rs

Lines 108 to 115 in 7dc6208

	sign: IfBlock::new::<()>(
	"auth.dkim.sign",
	[(
	"is_local_domain('*', sender_domain)",
	"['rsa-' + sender_domain, 'ed25519-' + sender_domain]",
	)],
	"false",
	),

Should automatically DKIM sign outgoing messages, however it does not seem to work. Only when setting the config manually to

[auth.dkim]
sign = ["'rsa-' + sender_domain", "'ed25519-' + sender_domain"]

it starts to work.

I suspect the is_local_domain is not correctly implemented. I'm using the memory directory for the account that sends the message.

How can we reproduce the problem?

Use default configs with the memory directory, send a message, check for DKIM signing status.

Version

v0.11.x or lower

What database are you using?

RocksDB

What blob storage are you using?

RocksDB

Where is your directory located?

None

What operating system are you using?

Linux

Relevant log output

Code of Conduct

• I agree to follow this project's Code of Conduct

[andreymal]

v0.11.x? 🧟

Well, I'm still using 0.10 and is_local_domain works fine