TIP: Use Markdown or, <pre> for multi line code blocks / <code> for inline code.
Set content-type for external PUT request?
  • FinbarrFinbarr

    Hello Kohana Community!

    Duplicate of: http://stackoverflow.com/questions/5252996/kohana-request-set-content-type

    I am having difficulty setting the content-type header of an external Kohana3.1+ request.

    $r = Request::factory($this->api) ->method(Request::PUT) ->body($this->to_json()) ->headers('content-type', 'application/json');

    The request ends up being executed by curl. The CURLOPT_ settings for the request type (PUT) and the request body (the json) are set correctly, but the content type does not get set.

    Am I going about this in the wrong way?

    I have debugged all the way down to Kohana_Request_Client_External class and at run time the $options variable in that class definitely gets the PUT and JSON, and it has a 3rd entry, but the value of that entry is the number 1.

    Any help much appreciated!

    Thanks, Finbarr

  • WayaWaya

    Did you try calling send_headers() with your request? Also I doubt it matters, but it might be better practice to upper case Content-Type header.

  • FinbarrFinbarr

    Which class has the send_headers() method?

  • WayaWaya

    If I'm not mistaken you can just chain it on to your request like: $r = Request::factory($this->api) ->method(Request::PUT) ->body($this->to_json()) ->headers('content-type', 'application/json')->send_headers();

  • shadowhandshadowhand

    Sending headers is done by the response, not the request. The request will automatically send headers when calling execute() for external requests.

  • FinbarrFinbarr

    Well it seems that the content-type header does not get sent.

  • DrPheltRightDrPheltRight 
    $response = Request::factory($this->api)
    ->method(Request::PUT)
    ->headers('content-type', 'application/json')
    ->body($this->to_json())
    ->execute();

    Does the above not work for you @Finbarr?

  • DarsstarDarsstar

    Somebody created this issue today...

  • FinbarrFinbarr

    @Darsstar I cherrypicked the fix attached to that issue and my problem is resolved.

Add a Comment