2210: Add input validation for domain creation r=mergify[bot] a=0pc0deFR
## What type of PR?
bug-fix
## What does this PR do?
This patch add the input validation for domain creation.
### Related issue(s)
- Mention an issue like: #1817
- Auto close an issue like: closes#1817
Co-authored-by: Kevin Falcoz <0pc0defr@gmail.com>
Co-authored-by: Dimitri Huisman <diman@huisman.xyz>
2275: Fix2274: Inbound emails are rejected r=mergify[bot] a=nextgens
## What type of PR?
bug-fix
## What does this PR do?
Fix email reception
### Related issue(s)
- closes#2274
## Prerequisites
No documentation as this affects only people running master since yesterday
Co-authored-by: Florent Daigniere <nextgens@freenetproject.org>
Co-authored-by: Florent Daigniere <nextgens@users.noreply.github.com>
2265: Prevent an exception from being thrown if the username isn't an email address r=mergify[bot] a=nextgens
## What type of PR?
enhancement
## What does this PR do?
Mailu expects users to identify using an email address; some brute-force script don't send just an username and this leads to an exception... that makes the error message and return code vary.
This PR prevents the exceptions from being thrown in the first place
```
WARNING in nginx: Invalid user 'xxxx': (builtins.ValueError) invalid email address (no "`@")"`
```
### Related issue(s)
- closes#2261
- closes#1750
Co-authored-by: Florent Daigniere <nextgens@freenetproject.org>
Co-authored-by: Florent Daigniere <nextgens@users.noreply.github.com>
2254: Send ISRG_X1 on port 25, make DANE pin that r=mergify[bot] a=nextgens
## What type of PR?
bug-fix
## What does this PR do?
Ensure we send ISRG_X1 in the handshake on port 25 (non-interactive, size doesn't really matter).
Update the DANE pin to reflect the change.
I am not sure whether we will need to add --preferred-chain= in the future; This may be the case when letsencrypt decides to use X2/the ECDSA chain
This needs to be tested on a letsencrypt account that isn't mine (I'm opted in for the alternate cert chains)
### Related issue(s)
- closes#2138
## Prerequisites
Before we can consider review and merge, please make sure the following list is done and checked.
If an entry in not applicable, you can check it or remove it from the list.
There's already a towncrier news for it
Co-authored-by: Florent Daigniere <nextgens@freenetproject.org>
2253: Workaround the infamous coredns feature r=mergify[bot] a=nextgens
## What type of PR?
bug-fix
## What does this PR do?
Ensure that we set the ``DO`` flag on our queries to work around https://github.com/coredns/coredns/issues/5189
Add a FAQ entry to point users in the right direction in other cases (dnsmasq), discourage users from running Mailu without unbound
### Related issue(s)
- closes#2243
- closes#2239
- #2164
- #2163
- #2162
- #2135
- #1988
## Prerequisites
Before we can consider review and merge, please make sure the following list is done and checked.
If an entry in not applicable, you can check it or remove it from the list.
- [x] In case of feature or enhancement: documentation updated accordingly
- [x] Unless it's docs or a minor change: add [changelog](https://mailu.io/master/contributors/workflow.html#changelog) entry file.
Co-authored-by: Florent Daigniere <nextgens@freenetproject.org>
Co-authored-by: Dimitri Huisman <diman@huisman.xyz>
Florent Daigniere