process headers before checking the content-type

Author: scrogson

lib/oauth2/response.ex

@@ -26,9 +26,10 @@ defmodule OAuth2.Response do
 
   @doc false
   def new(status_code, headers, body) do
+    headers = process_headers(headers)
     %__MODULE__{
       status_code: status_code,
-      headers: process_headers(headers),
+      headers: headers,
       body: decode_response_body(body, content_type(headers))
     }
   end
@@ -41,8 +42,6 @@ defmodule OAuth2.Response do
   defp decode_response_body(" ", _type), do: ""
   defp decode_response_body(body, "application/x-www-form-urlencoded"),
     do: URI.decode_query(body)
-  defp decode_response_body(body, "text/plain"),
-    do: URI.decode_query(body)
   defp decode_response_body(body, type) do
     if serializer = Application.get_env(:oauth2, :serializers)[type] do
       serializer.decode!(body)

test/oauth2/access_token_test.exs

@@ -10,7 +10,7 @@ defmodule OAuth2.AccessTokenTest do
   end
 
   test "new with 'expires' param" do
-    response = Response.new(200, [{"content-type", "text/plain"}], "access_token=abc123&expires=123")
+    response = Response.new(200, [{"Content-Type", "application/x-www-form-urlencoded"}], "access_token=abc123&expires=123")
     token = AccessToken.new(response.body)
     assert token.access_token == "abc123"
     assert token.expires_at == 123