3248 Commits (6953ee6bdea79408d7b49b89e0dc5ad54c8f66db)
 

Author SHA1 Message Date
Ionut Filip 4df36bea91 Removed commented options 6 years ago
Ionut Filip 72b94d28f7 Modified PR template 6 years ago
Ionut Filip 8489bcb05e Minor changes to changelog doc 6 years ago
Ionut Filip e674a6fe3d Added docs for new changelog entries 6 years ago
Ionut Filip e548c4e531 Moved towncrier to project root 6 years ago
Ionut Filip 4a03e32882 Towncrieri implementation 6 years ago
bors[bot] 211136f9b1 Merge #934
934: Pass optional arguments to domain model r=mergify[bot] a=ionutfilip

## What type of PR?
bug-fix

## What does this PR do?
Passing optional args to domain model when using cli.

### Related issue(s)
Fixes #933

## 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: Ionut Filip <ionut.philip@gmail.com>
6 years ago
bors[bot] d129733fac Merge #919
919: Install bash in alpine based images. r=mergify[bot] a=firvida

This fixes #918

Bash shell is used by default in Kubernetes' dashboard console, which is very
useful for admins.

## What type of PR?

bug-fix

## What does this PR do?

### Related issue(s)
- closes #918 

## 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: Abel Alfonso Fírvida Donéstevez <abel@merchise.org>
6 years ago
bors[bot] c07759dcc5 Merge #877
877: fix some cli commands in the docs r=mergify[bot] a=hoellen

## What type of PR?
documentation

## What does this PR do?
Since we updated the python package click to 7.0, commands with underscores in name are replaced by a dash. This PR fixes this in the documentation. 
It also fixes two `python manage.py [...]` leftovers. 

In addition I think we should rename the arguments now as well and replace underscores with dashes, e.g. `--max_users` to `--max-users`. So we keep it consistent. What do you think? I'd update this PR if you agree on this. 


### Related issue(s)
closes #849
closes #925

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



## Todo
- [x] #911 
- [x] #926 
- [x] rename options (`_` to `-`)
- [x] domain command


Co-authored-by: hoellen <dev@hoellen.eu>
Co-authored-by: Roman Hoellen <info@hoellen.eu>
Co-authored-by: hoellen <hoellen@users.noreply.github.com>
6 years ago
hoellen 62511b723b
Merge branch 'master' into fix-doc-commands-1 6 years ago
Ionut Filip 4d2844c1f5 Fixed arguments passed to domain 6 years ago
bors[bot] 626b8a9d05 Merge #929
929: Fix migration typo r=mergify[bot] a=muhlemmer

## What type of PR?

bug-fix

## What does this PR do?

Fix a typo in one of the migration scripts.

### Related issue(s)
- closes #905

## 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] Unless it's docs or a minor change: place entry in the [changelog](CHANGELOG.md), under the latest un-released version.


Co-authored-by: Tim Möhlmann <muhlemmer@gmail.com>
6 years ago
Ionut Filip f86e054d27 Add changelog entry 6 years ago
Roman Hoellen 9dbe623c18 update changelog 6 years ago
Roman Hoellen 1dfef1609a add domain cli to docs 6 years ago
Ionut Filip cebc64a280 Resolve HOST_WEBMAIL in admin 6 years ago
Roman Hoellen bc617cf8ec replace _ with - in cli options 6 years ago
Roman Hoellen ab33ba2172 Merge remote-tracking branch 'upstream/master' into fix-doc-commands-1 6 years ago
bors[bot] 6a408f6ac7 Merge #923
923: Update command for user import r=mergify[bot] a=zoldar

## What type of PR?

Documentation update.

## What does this PR do?

Updates obsolete user import command example.

## Related issue(s)

Refs https://github.com/Mailu/Mailu/issues/849
Refs https://github.com/Mailu/Mailu/pull/877#discussion_r255300087

## 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: Adrian Gruntkowski <adrian.gruntkowski@gmail.com>
6 years ago
Tim Möhlmann c1485f9381
Update CHANGELOG.md 6 years ago
Tim Möhlmann 111dae351a
Fix typo in migration file
Fixes #905
6 years ago
bors[bot] d3a5c70589 Merge #907
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>
6 years ago
Manuel Vázquez Acosta 6cc6f2742b
Activate ClamAV in filters tests.
See https://github.com/Mailu/Mailu/pull/907#issuecomment-462043036
6 years ago
Adrian Gruntkowski 3cfaa00fac
Update command for user import
The current example seems to be obsolete.
6 years ago
Dario Ernst 328db51032 Add FAQ about clients on port 25 generating non-helpful errors 6 years ago
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.
6 years ago
bors[bot] 86b4242f82 Merge #886
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 #844
- Fixes #827 
- Hopefully helps with #829 and #834

## 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>
6 years ago
Manuel Vázquez Acosta 27bbab97d6
Add Changlog entry. 6 years ago
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.
6 years ago
bors[bot] 4d58b322bc Merge #888 #889
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 #885

## 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 #856

## 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>
6 years ago
Dario Ernst ef5a114cff Put webmail on / for empty webmail_path from setup 6 years ago
Dario Ernst 0676252ec1 Revert accidental setup commits and fix code/pre tags 6 years ago
bors[bot] 702e8b384b Merge #882
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  #817 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>
6 years ago
Tim Möhlmann 06457e9617
Merge branch 'master' into fix_fetches_create 6 years ago
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 #856
6 years ago
cr1st1p 9b65a6c862
Merge branch 'master' into admin_create_user_enhancement 6 years ago
Dario Ernst 392637e16a Correct the URL users are directed to after using setup without exposing /admin
closes #885
6 years ago
bors[bot] 77ec049edc Merge #876
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 #869 

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

bors r+

Co-authored-by: Tim Möhlmann <muhlemmer@gmail.com>
6 years ago
Tim Möhlmann a731e04670
Update docs on review dissmissal and automatic rebuilds 6 years ago
hoellen 928b5470d4 fix some cli commands in the docs 6 years ago
hoellen fba246e9a1 add wildcard option to cli (alias) 6 years ago
bors[bot] 343ae8be49 Merge #851
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 #847 

## 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>
6 years ago
Tim Möhlmann 3f4c65e6d1 Make mergify shut-up when merged or closed 6 years ago