clone: use cgit_print_error_page() instead of html_status()
This provides a formatted error response rather than a simple HTTP error. Signed-off-by: John Keeping <john@keeping.me.uk>
This commit is contained in:
parent
e9b71ae6fe
commit
329381dfe4
10
ui-clone.c
10
ui-clone.c
@ -57,13 +57,13 @@ static void send_file(const char *path)
|
|||||||
if (stat(path, &st)) {
|
if (stat(path, &st)) {
|
||||||
switch (errno) {
|
switch (errno) {
|
||||||
case ENOENT:
|
case ENOENT:
|
||||||
html_status(404, "Not found", 0);
|
cgit_print_error_page(404, "Not found", "Not found");
|
||||||
break;
|
break;
|
||||||
case EACCES:
|
case EACCES:
|
||||||
html_status(403, "Forbidden", 0);
|
cgit_print_error_page(403, "Forbidden", "Forbidden");
|
||||||
break;
|
break;
|
||||||
default:
|
default:
|
||||||
html_status(400, "Bad request", 0);
|
cgit_print_error_page(400, "Bad request", "Bad request");
|
||||||
}
|
}
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
@ -78,7 +78,7 @@ static void send_file(const char *path)
|
|||||||
void cgit_clone_info(void)
|
void cgit_clone_info(void)
|
||||||
{
|
{
|
||||||
if (!ctx.qry.path || strcmp(ctx.qry.path, "refs")) {
|
if (!ctx.qry.path || strcmp(ctx.qry.path, "refs")) {
|
||||||
html_status(400, "Bad request", 0);
|
cgit_print_error_page(400, "Bad request", "Bad request");
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -91,7 +91,7 @@ void cgit_clone_info(void)
|
|||||||
void cgit_clone_objects(void)
|
void cgit_clone_objects(void)
|
||||||
{
|
{
|
||||||
if (!ctx.qry.path) {
|
if (!ctx.qry.path) {
|
||||||
html_status(400, "Bad request", 0);
|
cgit_print_error_page(400, "Bad request", "Bad request");
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Caricamento…
Fai riferimento in un nuovo problema
Block a user