Merge pull request 'dev' (#37) from dev into main
All checks were successful
Tests / CI / Run tests passed: 206
expenses_manager/pipeline/head This commit looks good

Reviewed-on: #37
This commit is contained in:
jkuijperm 2026-09-30 14:09:54 +00:00
commit f2e332f35b

View File

@ -212,17 +212,19 @@ pipeline {
'''
script {
// Confirm the application actually answers. A container that
// starts and immediately crashes would otherwise pass as a
// successful deployment.
// The Host header must match ALLOWED_HOSTS or Django answers
// 400 regardless of health. A 301 is the expected reply here:
// SECURE_SSL_REDIRECT sends plain HTTP to HTTPS, and reaching
// that redirect already proves the app processed the request.
def code = sh(returnStdout: true, script: '''
ssh ${SSH_OPTS} ${NAS_USER}@${NAS_HOST} \
"sleep 10 && curl -s -o /dev/null -w '%{http_code}' \
-H 'Host: finanzas.kuijper.es' \
http://127.0.0.1:8001/"
''').trim()
echo "Respuesta HTTP: ${code}"
if (!(code in ['200', '302'])) {
if (!(code in ['200', '301', '302'])) {
error("La aplicacion no responde correctamente (HTTP ${code})")
}
}