Skip to content

Commit 5d7278e

Browse files
committed
fix(mda): remove conflicting vendor files from configuration
1 parent 7e4f612 commit 5d7278e

4 files changed

Lines changed: 19 additions & 4 deletions

File tree

.github/linters/super-linter.env

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,10 +5,13 @@ VALIDATE_CHECKOV=false
55
VALIDATE_GITHUB_ACTIONS=false
66
VALIDATE_GITHUB_ACTIONS_ZIZMOR=false
77
VALIDATE_GIT_COMMITLINT=false
8+
VALIDATE_JAVASCRIPT_ES=false
9+
VALIDATE_JAVASCRIPT_PRETTIER=false
810
VALIDATE_JSCPD=false
911
VALIDATE_KUBERNETES_KUBECONFORM=false
1012
VALIDATE_PHP=false
1113
VALIDATE_PHP_PHPCS=false
14+
VALIDATE_PHP_PHPSTAN=false
1215
VALIDATE_PHP_PSALM=false
1316
VALIDATE_PYTHON_BLACK=false
1417
VALIDATE_SQLFLUFF=false

Makefile

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,8 @@ up: .env
4141

4242
.PHONY: fixtures
4343
fixtures:
44-
$(COMPOSE_PRODUCTION) run --rm web /opt/admin/bin/console system:check --wait
44+
$(COMPOSE_PRODUCTION) exec web /opt/admin/bin/console system:check --wait
45+
sleep 5 # TODO: remove when admin implemented better checks
4546
$(COMPOSE_PRODUCTION) exec web /opt/admin/bin/console domain:add example.com
4647
$(COMPOSE_PRODUCTION) exec web /opt/admin/bin/console domain:add example.org
4748
$(COMPOSE_PRODUCTION) exec web /opt/admin/bin/console user:add --admin --password=changeme --enable admin example.com
@@ -101,7 +102,7 @@ kubernetes-logs:
101102
kubernetes-test:
102103
kubectl delete -f test/k8s/test-job.yaml --ignore-not-found
103104
kubectl apply -f test/k8s/test-job.yaml
104-
kubectl wait --timeout=10m --for=condition=complete job -l app.kubernetes.io/name=test-runner-job
105+
kubectl wait --timeout=5m --for=condition=complete job -l app.kubernetes.io/name=test-runner-job
105106
kubectl logs --ignore-errors -l app.kubernetes.io/name=test-runner-job
106107

107108
.PHONY: kubernetes-up

target/mda/Dockerfile

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,11 @@ LABEL de.ressourcenkonflikt.docker-mailserver.autoheal="true"
88
USER root
99
RUN apt-get update && \
1010
apt-get install --no-install-recommends -y curl && \
11-
rm -rf /var/lib/apt/lists/*
11+
rm -r \
12+
/etc/dovecot/conf.d/mail.conf \
13+
/etc/dovecot/conf.d/ssl.conf \
14+
/etc/dovecot/ssl \
15+
/var/lib/apt/lists/*
1216
USER vmail
1317

1418
ENV MYSQL_HOST=db \

test/bats/integration/010_tls.bats

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,15 @@
44
[ -f /media/tls/tls.crt ]
55
}
66

7+
@test "compare certificate fingerprints" {
8+
MDA_FINGERPRINT=$(echo | openssl s_client -showcerts -connect "${MDA_IMAPS_ADDRESS}" 2>&1 | openssl x509 -fingerprint -noout)
9+
MTA_FINGERPRINT=$(echo | openssl s_client -showcerts -connect "${MTA_SMTP_ADDRESS}" -starttls smtp 2>&1 | openssl x509 -fingerprint -noout)
10+
11+
[ "$MDA_FINGERPRINT" = "$MTA_FINGERPRINT" ]
12+
}
13+
714
@test "connection to imaps" {
8-
run bash -c 'echo -e "a1 LOGOUT\r\n" | openssl s_client -quiet -brief -connect ${MDA_IMAPS_ADDRESS}'
15+
run bash -c 'echo -e "a1 LOGOUT\r\n" | openssl s_client -showcerts -quiet -brief -connect ${MDA_IMAPS_ADDRESS}'
916
[ "$status" -eq 0 ]
1017
}
1118

0 commit comments

Comments
 (0)