charset=utf-8

Closes https://github.com/gleam-wisp/wisp/issues/86

CHANGELOG.md

···
 # Changelog
 
+## v0.16.0 - Unreleased
+
+- HTML and JSON body functions now include `charset=utf-8` in the content-type
+  header.
+
 ## v0.15.0 - 2024-05-12
 
 - The `mist` version constraint has been increased to >= 1.2.0.

src/wisp.gleam

···
 /// ```
 ///
 pub fn html_response(html: StringBuilder, status: Int) -> Response {
-  HttpResponse(status, [#("content-type", "text/html")], Text(html))
+  HttpResponse(
+    status,
+    [#("content-type", "text/html; charset=utf-8")],
+    Text(html),
+  )
 }
 
 /// Create a JSON response.
···
 /// ```
 ///
 pub fn json_response(json: StringBuilder, status: Int) -> Response {
-  HttpResponse(status, [#("content-type", "application/json")], Text(json))
+  HttpResponse(
+    status,
+    [#("content-type", "application/json; charset=utf-8")],
+    Text(json),
+  )
 }
 
 /// Set the body of a response to a given HTML document, and set the
···
 /// let body = string_builder.from_string("<h1>Hello, Joe!</h1>")
 /// response(201)
 /// |> html_body(body)
-/// // -> Response(201, [#("content-type", "text/html")], Text(body))
+/// // -> Response(201, [#("content-type", "text/html; charset=utf-8")], Text(body))
 /// ```
 ///
 pub fn html_body(response: Response, html: StringBuilder) -> Response {
   response
   |> response.set_body(Text(html))
-  |> response.set_header("content-type", "text/html")
+  |> response.set_header("content-type", "text/html; charset=utf-8")
 }
 
 /// Set the body of a response to a given JSON document, and set the
···
 /// let body = string_builder.from_string("{\"name\": \"Joe\"}")
 /// response(201)
 /// |> json_body(body)
-/// // -> Response(201, [#("content-type", "application/json")], Text(body))
+/// // -> Response(201, [#("content-type", "application/json; charset=utf-8")], Text(body))
 /// ```
 ///
 pub fn json_body(response: Response, json: StringBuilder) -> Response {
   response
   |> response.set_body(Text(json))
-  |> response.set_header("content-type", "application/json")
+  |> response.set_header("content-type", "application/json; charset=utf-8")
 }
 
 /// Set the body of a response to a given string builder.
···
         |> result.unwrap("")
         |> marceau.extension_to_mime_type
 
+      let content_type = case mime_type {
+        "application/json" | "text/" <> _ -> mime_type <> "; charset=utf-8"
+        _ -> mime_type
+      }
+
       case simplifile.is_file(path) {
         Ok(True) ->
           response.new(200)
-          |> response.set_header("content-type", mime_type)
+          |> response.set_header("content-type", content_type)
           |> response.set_body(File(path))
         _ -> handler()
       }

test/wisp_test.gleam

···
   response.status
   |> should.equal(201)
   response.headers
-  |> should.equal([#("content-type", "application/json")])
+  |> should.equal([#("content-type", "application/json; charset=utf-8")])
   response
   |> testing.string_body
   |> should.equal("{\"one\":1,\"two\":2}")
···
   response.status
   |> should.equal(200)
   response.headers
-  |> should.equal([#("content-type", "text/html")])
+  |> should.equal([#("content-type", "text/html; charset=utf-8")])
   response
   |> testing.string_body
   |> should.equal("Hello, world!")
···
   response.status
   |> should.equal(405)
   response.headers
-  |> should.equal([#("allow", "GET"), #("content-type", "text/html")])
+  |> should.equal([
+    #("allow", "GET"),
+    #("content-type", "text/html; charset=utf-8"),
+  ])
   response
   |> testing.string_body
   |> should.equal("Hello, world!")
···
   response.status
   |> should.equal(200)
   response.headers
-  |> should.equal([#("content-type", "text/plain")])
+  |> should.equal([#("content-type", "text/plain; charset=utf-8")])
   response.body
   |> should.equal(wisp.File("./test/fixture.txt"))
 
···
   response.status
   |> should.equal(200)
   response.headers
-  |> should.equal([#("content-type", "application/json")])
+  |> should.equal([#("content-type", "application/json; charset=utf-8")])
   response.body
   |> should.equal(wisp.File("./test/fixture.json"))
 
+  // Get some other file
+  let response =
+    testing.get("/stuff/test/fixture.dat", [])
+    |> handler
+  response.status
+  |> should.equal(200)
+  response.headers
+  |> should.equal([#("content-type", "application/octet-stream")])
+  response.body
+  |> should.equal(wisp.File("./test/fixture.dat"))
+
   // Get something not handled by the static file server
   let response =
     testing.get("/stuff/this-does-not-exist", [])
···
   response.status
   |> should.equal(200)
   response.headers
-  |> should.equal([#("content-type", "text/plain")])
+  |> should.equal([#("content-type", "text/plain; charset=utf-8")])
   response.body
   |> should.equal(wisp.File("./test/fixture.txt"))
 }
···
   response.status
   |> should.equal(200)
   response.headers
-  |> should.equal([#("content-type", "text/plain")])
+  |> should.equal([#("content-type", "text/plain; charset=utf-8")])
   response.body
   |> should.equal(wisp.File("./test/fixture.txt"))
 }
···
   |> handler(Error(Nil))
   |> should.equal(Response(
     201,
-    [#("content-type", "text/html")],
+    [#("content-type", "text/html; charset=utf-8")],
     wisp.Text(string_builder.from_string("Hello!")),
   ))
 
   testing.get("/", [])
   |> request.set_method(http.Head)
   |> handler(Ok("HEAD"))
-  |> should.equal(Response(201, [#("content-type", "text/html")], wisp.Empty))
+  |> should.equal(Response(
+    201,
+    [#("content-type", "text/html; charset=utf-8")],
+    wisp.Empty,
+  ))
 
   testing.get("/", [])
   |> request.set_method(http.Post)
···
   |> wisp.json_body(string_builder.from_string("{\"one\":1,\"two\":2}"))
   |> should.equal(Response(
     200,
-    [#("content-type", "application/json")],
+    [#("content-type", "application/json; charset=utf-8")],
     wisp.Text(string_builder.from_string("{\"one\":1,\"two\":2}")),
   ))
 }