-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding the PERMIT_DOCKER option #270
Conversation
See README.md for more informations
Job restarted. |
You should add functional tests (have a look to |
Done |
@test "checking PERMIT_DOCKER: mynetworks option exist" { | ||
run docker exec mail /bin/sh -c "postconf -d | grep '^mynetworks ='" | ||
[ "$status" -eq 0 ] | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will be always true because a default value exists in the main.cf
The test should be the existing one, testing different value of the variable (empty, host, network). |
Otherwise the coverage remains low. |
Ok I will make some test with the different containers.
|
Fix docker network range Adding opendkim and openmarc configuration
@tomav The Travis build is blocked again. |
Done |
Thank you @Zehir, great enhancement! |
* Adding the PERMIT_DOCKER option See README.md for more informations * Adding some test for PERMIT_DOCKER option * Fix test cases * Opendkim and Openmarc configuration Fix docker network range Adding opendkim and openmarc configuration * Adding some options for tests * Update log message * Update tests
Sorry if the question is stupid guys, What would be the recipe for disabling all SMTP attmpts except those within
😕 |
* Adding the PERMIT_DOCKER option See README.md for more informations * Adding some test for PERMIT_DOCKER option * Fix test cases * Opendkim and Openmarc configuration Fix docker network range Adding opendkim and openmarc configuration * Adding some options for tests * Update log message * Update tests
See README.md for more informations
Enhancement #261