Send a charset only with a text type

Archives, packs and the HEAD file went out as octet-stream or gzip
with a UTF-8 charset on the end, since only the blob page cleared it.
The headers now decide from the type itself, so the two places that
cleared the charset by hand no longer need to.

3 files changed, 14 insertions, 10 deletions

diff --git a/source/ui-plain.c b/source/ui-plain.c
index 7db6e07..6d02051 100644
--- a/source/ui-plain.c
+++ b/source/ui-plain.c
@@ -81,14 +81,8 @@ static void print_object(const struct object_id *oid, const char *path)
 			ctx.page.mimetype = NULL;
 	}
 
-	if (!ctx.page.mimetype) {
-		if (buffer_is_binary(buf, size)) {
-			ctx.page.mimetype = "application/octet-stream";
-			ctx.page.charset = NULL;
-		} else {
-			ctx.page.mimetype = "text/plain";
-		}
-	}
+	if (!ctx.page.mimetype)
+		ctx.page.mimetype = buffer_is_binary(buf, size) ? "application/octet-stream" : "text/plain";
 	ctx.page.filename = path;
 	ctx.page.size = size;
 	cgit_print_http_headers();
diff --git a/source/ui-shared.c b/source/ui-shared.c
index 3970c58..a112fb2 100644
--- a/source/ui-shared.c
+++ b/source/ui-shared.c
@@ -1424,6 +1424,15 @@ void cgit_print_age(time_t t, int tz, time_t max_relative)
 	print_rel_date(t, tz, secs * 1.0 / SECONDS_PER_YEAR, "age-years", "years");
 }
 
+/*
+ * Whether a content type carries text a charset applies to, which is every
+ * text type and the XML ones, the atom feed among them.
+ */
+static int type_has_charset(const char *mimetype)
+{
+	return starts_with(mimetype, "text/") || strstr(mimetype, "xml");
+}
+
 void cgit_print_http_headers(void)
 {
 	ctx.page.headers_sent = 1;
@@ -1436,7 +1445,9 @@ void cgit_print_http_headers(void)
 		htmlf("Status: %d %s\n", ctx.page.status, ctx.page.statusmsg);
 	else
 		html("Status: 200 OK\n");
-	if (ctx.page.mimetype && ctx.page.charset)
+	// A charset only means something for text, so an archive or a pack
+	// is sent without one.
+	if (ctx.page.mimetype && ctx.page.charset && type_has_charset(ctx.page.mimetype))
 		htmlf("Content-Type: %s; charset=%s\n", ctx.page.mimetype, ctx.page.charset);
 	else if (ctx.page.mimetype)
 		htmlf("Content-Type: %s\n", ctx.page.mimetype);
diff --git a/source/ui-summary.c b/source/ui-summary.c
index eb04ecd..61cd6ac 100644
--- a/source/ui-summary.c
+++ b/source/ui-summary.c
@@ -138,7 +138,6 @@ void cgit_print_repo_readme(const char *path)
 	mimetype = cgit_get_mimetype_for_filename(path);
 	if (mimetype && (starts_with(mimetype, "image/") || starts_with(mimetype, "video/"))) {
 		ctx.page.mimetype = mimetype;
-		ctx.page.charset = NULL;
 		cgit_print_plain();
 		free(mimetype);
 		return;