mirror of
https://blitiri.com.ar/repos/chasquid
synced 2026-06-09 19:33:35 +00:00
Alberto Bertogli
ab2a6e7eb3
In Go 1.25.11/1.26.4, net/textproto error formatting was changed, and now the message is quoted. This is to protect from general string injections. See https://go-review.googlesource.com/c/go/+/778182 and ec1c380418ec6a0da28d4519872e2b81ba9152ba. After this change, the error strings from net/textproto (and net/smtp which uses it) now include quotes. For chasquid, the only user-observable change in the server is that the errors included in the DSN messages are now quoted, which should not be a problem. However, some of the tests which validate error strings need to be adjusted. The patch also updates go.mod to set the Go version to match this new minimum.
81 lines
2.0 KiB
Bash
Executable File
81 lines
2.0 KiB
Bash
Executable File
#!/bin/bash
|
|
|
|
set -e
|
|
. "$(dirname "$0")/../util/lib.sh"
|
|
|
|
init
|
|
check_hostaliases
|
|
|
|
generate_certs_for testserver
|
|
add_user user@testserver secretpassword
|
|
add_user someone@testserver secretpassword
|
|
add_user blockme@testserver secretpassword
|
|
add_user permanent@testserver secretpassword
|
|
|
|
mkdir -p .logs
|
|
chasquid -v=2 --logfile=.logs/chasquid.log --config_dir=config &
|
|
wait_until_ready 1025
|
|
|
|
cp config/hooks/post-data.good config/hooks/post-data
|
|
|
|
smtpc someone@testserver < content
|
|
|
|
wait_for_file .mail/someone@testserver
|
|
|
|
mail_diff content .mail/someone@testserver
|
|
|
|
if ! grep -q "X-Post-Data: success" .mail/someone@testserver; then
|
|
fail "missing X-Post-Data header"
|
|
fi
|
|
|
|
function check() {
|
|
if ! grep -q "$1" .data/post-data.out; then
|
|
fail "missing: $1"
|
|
fi
|
|
}
|
|
|
|
# Verify that the environment for the hook was reasonable.
|
|
check "RCPT_TO=someone@testserver"
|
|
check "MAIL_FROM=user@testserver"
|
|
check "USER=$USER"
|
|
check "PWD=$PWD/config"
|
|
check "EHLO_DOMAIN=localhost"
|
|
check "EHLO_DOMAIN_RAW=localhost"
|
|
check "FROM_LOCAL_DOMAIN=1"
|
|
check "ON_TLS=1"
|
|
check "AUTH_AS=user@testserver"
|
|
check "PATH="
|
|
check "REMOTE_ADDR="
|
|
check "SPF_PASS=0"
|
|
|
|
|
|
# Check that failures in the script result in failing delivery.
|
|
# Transient failure.
|
|
if smtpc blockme@testserver < content >.logs/smtpc.log 2>&1; then
|
|
fail "ERROR: hook did not block email as expected"
|
|
fi
|
|
if ! grep -q '451 "¡No pasarán!"' .logs/smtpc.log; then
|
|
cat .logs/smtpc.log
|
|
fail "ERROR: transient hook error not returned correctly"
|
|
fi
|
|
|
|
# Permanent failure.
|
|
if smtpc permanent@testserver < content >.logs/smtpc.log 2>&1; then
|
|
fail "ERROR: hook did not block email as expected"
|
|
fi
|
|
if ! grep -q '554 "Nos hacemos la permanente"' .logs/smtpc.log; then
|
|
cat .logs/smtpc.log
|
|
fail "ERROR: permanent hook error not returned correctly"
|
|
fi
|
|
|
|
# Check that the bad hooks don't prevent delivery.
|
|
for i in config/hooks/post-data.bad*; do
|
|
cp "$i" config/hooks/post-data
|
|
|
|
smtpc someone@testserver < content
|
|
wait_for_file .mail/someone@testserver
|
|
mail_diff content .mail/someone@testserver
|
|
done
|
|
|
|
success
|