Fixing case for x-jwt-token

[rmckeel]

Response headers come back with case sensitive "X-Jwt-Token". The field "X-Jwt-Token" seems to be case sensitive. That is, checking $response['headers']['x-jwt-token'] does not work, whereas checking $response['headers']['X-Jwt-Token'] does work.

Correcting case sensitivity issue in this codebase.

[travist]

Hey Ryan,

Thanks for your contribution. I would prefer to have it so that it is case insensitive so that we don't possibly break other cases. I would propose making the following change to the getHeaders method. Let me know if this works.

private function getHeaders($header) {
    $headers = array();
    foreach (explode("\r\n", $header) as $i => $line) {
      if ($i === 0) {
        $headers['http_code'] = $line;
      }
      else {
        list ($key, $value) = explode(': ', $line);
        $headers[strtolower($key)] = $value;
      }
    }
    return $headers;
  }

Notice the strtolower when assigning the key.

[rmckeel]

Travis, that works great! That is more resilient than what I was thinking.

I was thinking about adding some PHP documentation and a little formatting. Does that seem reasonable, or do you already have it in a format and doc style you like?

[rmckeel]

Creating new pull request with minimal changes, except strtolower change for response header alignment.