Use x-ndjson

phoenixframework · Feb 17, 2023 · bd3ce36 · bd3ce36
1 parent 36b2c16
commit bd3ce36
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/assets/js/phoenix/longpoll.js b/assets/js/phoenix/longpoll.js
@@ -123,7 +123,7 @@ export default class LongPoll {
 
   batchSend(messages){
     this.awaitingBatchAck = true
-    this.ajax("POST", "application/ndjson", messages.join("\n"), () => this.onerror("timeout"), resp => {
+    this.ajax("POST", "application/x-ndjson", messages.join("\n"), () => this.onerror("timeout"), resp => {
       this.awaitingBatchAck = false
       if(!resp || resp.status !== 200){
         this.onerror(resp && resp.status)

diff --git a/lib/phoenix/transports/long_poll.ex b/lib/phoenix/transports/long_poll.ex
@@ -75,7 +75,7 @@ defmodule Phoenix.Transports.LongPoll do
         # we need to match on both v1 and v2 protocol, as well as wrap for backwards compat
         batch =
           case get_req_header(conn, "content-type") do
-            ["application/ndjson"] -> String.split(body, ["\n", "\r\n"])
+            ["application/x-ndjson"] -> String.split(body, ["\n", "\r\n"])
             _ -> [body]
           end
 

diff --git a/test/phoenix/integration/long_poll_channels_test.exs b/test/phoenix/integration/long_poll_channels_test.exs
@@ -161,7 +161,7 @@ defmodule Phoenix.Integration.LongPollChannelsTest do
     {serializer, json} = serializer(vsn, json)
     headers =
       if is_list(json) do
-        Map.merge(%{"content-type" => "application/ndjson"}, headers)
+        Map.merge(%{"content-type" => "application/x-ndjson"}, headers)
       else
         Map.merge(%{"content-type" => "application/json"}, headers)
       end