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;