Commit Graph

2328 Commits (7b27d0dd4ef68ed8cf82bd7141b0b677fa43ba19)
 

Author SHA1 Message Date
Tim Möhlmann 111dae351a
Fix typo in migration file
Fixes 
bors[bot] d3a5c70589 Merge
907: Don't generate the clamav configuration if ANTIVIRUS is none. r=mergify[bot] a=mvaled

## What type of PR?

bug-fix

## What does this PR do?

Avoid rspamd to try to connect to clamav if you choose ANTIVIRUS="none".  Otherwise the rspamd will try to connect to none, timing out several times and leading to poor performance.

I have a server without CLAMAV; and I'm getting times up to 45s:

```
$ grep 'CLAMAV' ~/downloads/logs-from-antispam-in-mailu-security-5d75fb987-kv958.txt | grep -o 'time: [^m]*'
time: 45011.089
time: 45126.002
time: 45002.024
time: 45037.436
time: 45006.775
...
```

Mails for which clamav is not used range from a tens of miliseconds to a few hundred.


## Prerequistes
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.

- [ ] In case of feature or enhancement: documentation updated accordingly
- [ ] Unless it's docs or a minor change: place entry in the [changelog](CHANGELOG.md), under the latest un-released version.


Co-authored-by: Manuel Vázquez Acosta <manuel@merchise.org>
Manuel Vázquez Acosta 6cc6f2742b
Activate ClamAV in filters tests.
See https://github.com/Mailu/Mailu/pull/907#issuecomment-462043036
Adrian Gruntkowski 3cfaa00fac
Update command for user import
The current example seems to be obsolete.
Dario Ernst 328db51032 Add FAQ about clients on port 25 generating non-helpful errors
Abel Alfonso Fírvida Donéstevez 39444c794e Install bash in alpine based images.
This fix https://github.com/Mailu/Mailu/issues/918

Bash shell is used by default in Kubernetes' dashboard console, which is very
useful for admins.
bors[bot] 86b4242f82 Merge
886: Ipv6 support r=mergify[bot] a=muhlemmer

## What type of PR?

(Feature, enhancement, bug-fix, documentation) -> A bit of everything

## What does this PR do?

Document how to use ipv6nat. This, however triggers some kind of flaky behavior with the Docker DNS resolver, resulting in lookup failures between containers.  So all resolving needs to be done during container startup/configuration.

In order not to pollute every single start.py file, we've created a small library called [Mailu/MailuStart](https://github.com/Mailu/MailuStart). As an addition, this library also defines the template generation function, including its logging facility.

Note: `docker-compose.yml` downgrade is necessary, as IPv6 settings are not supported by the Docker Compose file format 3 😞  

### Related issue(s)
Supersedes  PR 
- Fixes  
- Hopefully helps with  and 

## No backport yet

This PR directly imports MailuStart from git. This makes it a bit more simple to implement on the short term an do some testing and probably some future improvements. When everything is proved stable, we will create a proper PyPi package with versioning and consider back porting.

## Prerequistes
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: place entry in the [changelog](CHANGELOG.md), under the latest un-released version.


Co-authored-by: Ionut Filip <ionut.philip@gmail.com>
Co-authored-by: Tim Möhlmann <muhlemmer@gmail.com>
Manuel Vázquez Acosta 27bbab97d6
Add Changlog entry.
Manuel Vázquez Acosta 611363cbe4
Don't generate the clamav configuration if ANTIVIRUS is none.
Otherwise the rspamd will try to connect to none, timing out several times and
leading to poor performance.
bors[bot] 4d58b322bc Merge
888: Correct the URL users are directed to after using setup without exposing /admin r=mergify[bot] a=Nebukadneza

In setup, users who didn’t expose `/admin` were directed to `http://127.0.0.1:8080/`, which was missing `/ui` to work.

Also, let’s help out the less apt admins by providing a copy-paste template for ssh-tunnelling.

## What type of PR?
documentation

### Related issue(s)
closes 

## Prerequistes
- [x] In case of feature or enhancement: documentation updated accordingly
- [x] Unless it's docs or a minor change: place entry in the [changelog](CHANGELOG.md), under the latest un-released version.


889: Catch webmail-url empty but webmail configured and force to default r=mergify[bot] a=Nebukadneza

## What type of PR?
bug-fix

## What does this PR do?
Quite some users managed to delete the contents of the webmail-url field in
setup, which forces front into a restart loop. Catch the case where a webmail
service is configured, but url is empty — and force to default /webmail.

### Related issue(s)
closes 

## Prerequistes
- [x] In case of feature or enhancement: documentation updated accordingly
- [x] Unless it's docs or a minor change: place entry in the [changelog](CHANGELOG.md), under the latest un-released version.


Co-authored-by: Dario Ernst <dario@kanojo.de>
Dario Ernst ef5a114cff Put webmail on / for empty webmail_path from setup
Dario Ernst 0676252ec1 Revert accidental setup commits and fix code/pre tags
bors[bot] 702e8b384b Merge
882: Fix password validator for creating fetched accounts r=muhlemmer a=Nebukadneza

## What type of PR?
bug-fix

## What does this PR do?
Creating fetched accounts was changed in   with a slight typo. I’m an idiot since i approved, but only tested editing, and not creation. But here’s the fix!

### Related issue(s)
Small fix, didn’t feel explanatory issue was necessary

## Prerequistes
- [x] In case of feature or enhancement: documentation updated accordingly
  - Not needed
- [x] Unless it's docs or a minor change: place entry in the [changelog](CHANGELOG.md), under the latest un-released version.
  - Done


Co-authored-by: Dario Ernst <dario.ernst@rommelag.com>
Co-authored-by: Tim Möhlmann <muhlemmer@gmail.com>
Tim Möhlmann 06457e9617
Merge branch 'master' into fix_fetches_create
Dario Ernst d9d6f514aa Catch webmail-url empty but webmail configured and force to default
Quite some users managed to delete the contents of the webmail-url field in
setup, which forces front into a restart loop. Catch the case where a webmail
service is configured, but url is empty — and force to default /webmail.

closes 
cr1st1p 9b65a6c862
Merge branch 'master' into admin_create_user_enhancement
Dario Ernst 392637e16a Correct the URL users are directed to after using setup without exposing /admin
closes 
bors[bot] 77ec049edc Merge
876: Add wildcard aliases with the cli r=mergify[bot] a=hoellen

## What type of PR?
enhancement

## What does this PR do?
Add a possibility to add wildcard aliases with the cli.

### Related issue(s)
- closes  

## Prerequistes
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: place entry in the [changelog](CHANGELOG.md), under the latest un-released version.


Co-authored-by: hoellen <dev@hoellen.eu>
Ionut Filip f8dffe5a19
Resolve hosts in admin
Ionut Filip 004a431e97
Change to mailustart functions
cristi 244c78a17b Adding entry in Changelog
cristi cfbc38b956 Updated doc
Dario Ernst 34b31727c4 Fix password validator for creating fetched accounts
cristi a071181c61 Allow to automatically create admin account during startup.
bors[bot] d301de7d83 Merge
879: Update docs on review dissmissal and automatic rebuilds r=muhlemmer a=muhlemmer

bors r+

Co-authored-by: Tim Möhlmann <muhlemmer@gmail.com>
Tim Möhlmann a731e04670
Update docs on review dissmissal and automatic rebuilds
hoellen 928b5470d4 fix some cli commands in the docs
hoellen fba246e9a1 add wildcard option to cli (alias)
bors[bot] 343ae8be49 Merge
851: Make certdumper output fullchain-pems r=mergify[bot] a=Nebukadneza

## What type of PR?
bug-fix

## What does this PR do?
Before it only outputted a pem-file with the server-certificate —
however, it seems some clients want the fullchain delivered, as it’s
common with letsencrypt.

### Related issue(s)
closes  

## Prerequistes
- [x] In case of feature or enhancement: documentation updated accordingly
- [x] Unless it's docs or a minor change: place entry in the [changelog](CHANGELOG.md), under the latest un-released version.
  - minor fix

Co-authored-by: Dario Ernst <github@kanojo.de>
Tim Möhlmann 3f4c65e6d1 Make mergify shut-up when merged or closed
Dario Ernst 2c5f977117
Make certdumper output fullchain-pems
Before it only outputted a pem-file with the server-certificate —
however, it seems some clients want the fullchain delivered, as it’s
common with letsencrypt.

closes 
Tim Möhlmann 49192deec8
Merge pull request from usrpro/bors-ng
Automatic creation of review images
Tim Möhlmann a9d368d2c9
Fix typo in docs
Tim Möhlmann 33db705194
Use bors-ng to create and upload test images
- Reinstate Travis deploy phase
- Better labeling of Mergify rules
- Automatic `bors try` by Mergify
- Explain bors in comment message
- Skip push for staging branch
- Re-update docs to current situation
Ionut Filip 9684ebf33f Use mailustart package from git
Ionut Filip bdb0821e6e Merge branch 'feat-setup-ipv6' of github.com:usrpro/Mailu into feat-setup-ipv6
hoellen 808092c65e
Merge pull request from hoellen/token-ipv6
allow ipv6 addresses for tokens
Nebukadneza f050140be2
Merge pull request from merchise/issue-846
Check if sender address is not null before trying to check the domain.
Tim Möhlmann faca9983a8
Automatic creation of review images
- Enable bors-ng for better merging
- Re-enable mergify with a fixed up syntax, it will now only issue bors commands.
- Pull request build images are now pushed to docker hub
- Automatic backport rule
- Fix build tag for branches with slash (/)
- Only push when master and 1.6 target branch
Ionut Filip 782c686ea2
Merge pull request from steelcowboy/master
Fix environment variable inconsistency
Jim Heald eb8d84aed0 Fix environment variable inconsistency
Tim Möhlmann b46fd3bde4
Move changelog entry to 1.6.1
Tim Möhlmann e341e0141f
Merge remote-tracking branch 'upstream/master' into token-ipv6
Manuel Vázquez Acosta dfe5ddb18e
Check if sender address is not null before trying to check the domain.
Tim Möhlmann e80a7a1228
Merge pull request from HorayNarea/feat-unbound-drop-priv
unbound: drop privileges after binding to port
Tim Möhlmann 36a5c1c523
Merge pull request from bladeswords/patch-1
Updated doc to reflect 1.6 release
bladeswords 3f5d08235b
Updated doc to reflect 1.6 release
Thomas Sänger bb5852cc1c
add unbound priv-drop to changelog
Thomas Sänger f50995b5e6
unbound: drop privileges after binding to port
Tim Möhlmann de8e6f74ea
1.6 release notes ()
* 1.6 release!
* Fix typo in release notes
* Rectify Alpine 3.8.2 in release notes