Open bilogic opened 7 months ago
I had a quick look over the source code, seems easy. However, some questions as I'm not familiar with python
etc/mailinabox.conf
?~ Seems to be the env
variable.DKIM_SELECTOR=mail
as defaulttest_dns.py
and dns_update.py
, ok?I have some commits here https://github.com/mail-in-a-box/mailinabox/compare/main...bilogic:mailinabox:custom-dkim-selector, apologies that it is mixed in with a configurable TTL feature.
The main change is on line 792 of dns_update.py
where I replaced mail
with {selector}
My problem is, why does the /admin#external_dns
still show mail._domainkey...
? Where is the code to make the page reflect the custom selector?
mail
as a selector because there are commercial services using the same selector so there is a potential to be in conflict. Better would be either configurable selector or at least something unique.Then, ideally, this stimulates getting the PR merged.
Look at the dkim.sh script in the setup directory. There the default dns entry is generated. Because this is done only at initial setup, you need to include some logic to generate a new mail.txt on every run of the setup, or include generating the dns record in dns_update.py
Also, in start.sh you might want to use something like DKIM_SELECTOR=${DEFAULT_DKIM_SELECTOR:-mail} Your current code will always use mail as selector (and overwrite anything else)
Just finished a hardware migration. Thank you for the info, let me find some time to review them.
But 1 quick qn first, is the maintainer open to accepting dkimpy first?
python and bash are like my 5th language 🤣
@myfirstnameispaul
Definitely configurable (and not just something different). Since we are going to change it, let's provide an end-all solution. From the looks of it, this is not a very big change here, so if and when we switch to dkimpy, I would be happy to help.
@kiekerjan
start.sh
, but where does $DEFAULT_DKIM_SELECTOR
come from?~ All lines in an existing mailinbox.conf
are prefixed with DEFAULT_
Thanks!
My new branch with the key bits here https://github.com/mail-in-a-box/mailinabox/commit/b51550de1120ffdb63c299c64f6837004442de80
Please help me to review. Thank you.
Hi!
Is it possible to specify another DKIM selector other than
mail
?