Skip to content

Introduce ENABLE_AMAVIS env#1866

Merged
wernerfred merged 7 commits intodocker-mailserver:masterfrom
casperklein:amavis-patch
Mar 28, 2021
Merged

Introduce ENABLE_AMAVIS env#1866
wernerfred merged 7 commits intodocker-mailserver:masterfrom
casperklein:amavis-patch

Conversation

@casperklein
Copy link
Copy Markdown
Member

@casperklein casperklein commented Mar 24, 2021

Description

This allows to disable Amavis. This is useful, when ClamAV and Spamassassin are turned off.

Fixes #465

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Improvement (non-breaking change that does improve existing functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (README.md or ENVIRONMENT.md or the Wiki)
  • If necessary I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

@casperklein casperklein marked this pull request as ready for review March 25, 2021 00:57
@casperklein casperklein requested a review from a team March 25, 2021 00:58
@casperklein casperklein added area/documentation area/features area/scripts kind/new feature A new feature is requested in this issue or implemeted with this PR labels Mar 25, 2021
fbartels
fbartels previously approved these changes Mar 25, 2021
@georglauterbach
Copy link
Copy Markdown
Member

I'd leave it up to @wernerfred or @polarathene to merge this, as I do not know whether this may interfere with #1826. Currently, only rebasing onto current master would need to be done, but really I'm not sure.

The changes look good though!

@wernerfred
Copy link
Copy Markdown
Member

We will Take Care of it (probably at Weekend when Wiki pr gets merged)

Comment thread ENVIRONMENT.md Outdated
Comment thread mailserver.env Outdated
williamdes
williamdes previously approved these changes Mar 27, 2021
Co-authored-by: William Desportes <[email protected]>
Co-authored-by: William Desportes <[email protected]>
@wernerfred wernerfred merged commit dd0b399 into docker-mailserver:master Mar 28, 2021
@casperklein casperklein deleted the amavis-patch branch March 28, 2021 20:04
@georglauterbach
Copy link
Copy Markdown
Member

@all-contributors please add @casperklein for maintenance

@allcontributors
Copy link
Copy Markdown
Contributor

@georglauterbach

@casperklein already contributed before to maintenance

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/documentation area/features area/scripts kind/new feature A new feature is requested in this issue or implemeted with this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add ENABLE_AMAVIS env

5 participants