Change Content-Type header key to properly parse json response

[mationorato]

Hey, I found an error that was causing the callback response not to be properly decoded.

It seems that the Twitter API changes the content-typeresponse header to Content-Type. Causing the callback request handling to fail at detecting when was needed to decode the response body from json.

[yordis]

This shouldn't be the case since everything suppose to be normalized to downcased version. Could share more information and ideally create a unit test to prove such situation?

[mationorato]

Hey! Yes, I must admit that this looks weird to me too, specially since It is not the first time I'm using this library for a project, but the response.headers are not arriving in lowercase to

ueberauth_twitter/lib/ueberauth/strategy/twitter/internal.ex

Line 26 in baf1e8c

def decode_body({:ok, response}) do

.

Not sure it is easy to setup a test on this, But i can share a demo of whats going on in a few hours.

[mationorato]

in fact, nothing is being converted to lowercase here:
https://github.com/ueberauth/ueberauth_twitter/blob/baf1e8c022b6d074cf6d642fcb31d8470420201d/lib/ueberauth/strategy/twitter/internal.ex#L20C1-L20C92

[yordis]

HTTPoison ... this is old code ...