httpsrv: a limit exceeded answered with 503, not ignored
Some checks failed
C/C++ CI Linux / ${{ matrix.target }} (ubuntu-24.04-arm) (push) Has been cancelled
C/C++ CI Linux / ${{ matrix.target }} (ubuntu-latest) (push) Has been cancelled
C/C++ CI MacOS / ${{ matrix.target }} (macos-15) (push) Has been cancelled
C/C++ CI Windows / ${{ matrix.target }} (windows-2022) (push) Has been cancelled
C/C++ CI cmake / ${{ matrix.target }} (macos-15) (push) Has been cancelled
C/C++ CI cmake / ${{ matrix.target }} (ubuntu-24.04-arm) (push) Has been cancelled
C/C++ CI cmake / ${{ matrix.target }} (ubuntu-latest) (push) Has been cancelled
C/C++ CI cmake / ${{ matrix.target }} (windows-2022) (push) Has been cancelled
C/C++ CI cmake / ubuntu-latest (wolfSSL) (push) Has been cancelled

Authorization reports 10 when a limit of the service was exceeded,
which every other child answers with 503; the http server took it as
success and answered the request anyway. It now says 503 for itself,
after the body of the request is drained as any answer here does.
This commit is contained in:
Vladimir Dubrovin 2026-11-08 14:39:54 +03:00
parent c0eb7a65c6
commit 5a2e9d27c3

View File

@ -1476,16 +1476,27 @@ static int httpsrv_request(struct clientparam *param, struct httpreq *r)
return 1; return 1;
} }
} }
if(i && i != 10){ if(i){
/* The body goes before the answer, as for any answer this server /* The body goes before the answer, as for any answer this server
gives: left in the stream it would be read as the next request. */ gives: left in the stream it would be read as the next request. */
if(!r->drained){ if(!r->drained){
if(r->contentlen) httpsrv_drain(param, r->contentlen); if(r->contentlen) httpsrv_drain(param, r->contentlen);
r->drained = 1; r->drained = 1;
} }
/* 10: a limit of the service was exceeded, which the other
services answer with 503. */
if(i == 10){
static const char body[] = "exceeded your limits\n";
r->hdrs = NULL;
r->maxage = -1;
r->keepalive = 0;
if(httpsrv_head(r, 503, "text/plain", (int64_t)sizeof(body) - 1, NULL)) return 1;
httpsrv_send(r, body, (int)sizeof(body) - 1);
}
/* 4 no credentials, 5 unknown user, 6 wrong password: all of them /* 4 no credentials, 5 unknown user, 6 wrong password: all of them
should let the client offer credentials again. */ should let the client offer credentials again. */
if(i >= 4 && i <= 6) op_authrequired(r); else if(i >= 4 && i <= 6) op_authrequired(r);
else op_forbidden(r); else op_forbidden(r);
RETURN(i); RETURN(i);
} }