diff options
| author | Xavier Del Campo Romero <xavi.dcr@tutanota.com> | 2023-03-20 03:32:00 +0100 |
|---|---|---|
| committer | Xavier Del Campo Romero <xavi.dcr@tutanota.com> | 2023-03-20 10:57:20 +0100 |
| commit | d9bb874591c63f2efbfc1c4c953934251c700e9f (patch) | |
| tree | 88254b346628a22ece5a4ede5f411458969720c0 /main.c | |
| parent | d51b191ab7005de6679a00dd200ad5502ecff5ac (diff) | |
| download | slcl-d9bb874591c63f2efbfc1c4c953934251c700e9f.tar.gz | |
Send response on quota exceeded
So far, slcl would just close the connection with a client when the
Content-Length of an incoming request exceeded the user quota, without
any meaningful information given back to the user.
Now, slcl responds with a HTML file with meaningful information about
the error.
Limitations:
- While this commits has been successfully tested on ungoogled-chromium,
LibreWolf (and I assume Firefox and any other derivates too) does not
seem to receive the response from the server.
- However, this issue only occurred during local testing, but not
on remote instances.
Diffstat (limited to 'main.c')
| -rw-r--r-- | main.c | 14 |
1 files changed, 12 insertions, 2 deletions
@@ -610,7 +610,8 @@ static int check_quota(const struct auth *const a, const char *const username, } static int check_length(const unsigned long long len, - const struct http_cookie *const c, void *const user) + const struct http_cookie *const c, struct http_response *const r, + void *const user) { struct auth *const a = user; const char *const username = c->field; @@ -623,7 +624,16 @@ static int check_length(const unsigned long long len, return -1; } else if (has_quota) - return check_quota(a, username, len, quota); + { + int res = check_quota(a, username, len, quota); + + if (res < 0) + fprintf(stderr, "%s: check_quota failed\n", __func__); + else if (res > 0 && page_quota_exceeded(r, len, quota) < 0) + return -1; + + return res; + } return 0; } |
