Parse a POST request sent as content-type application/json? about grape HOT 6 CLOSED

This seems to work if you load Rack::PostBodyContentTypeParser from rack-contrib. Seems like something important to support

from grape.

Perhaps add a documentation patch to show how to include it?

from grape.

I've noticed this as well when sending POST requests from Safari, but everything works from Chrome. The only difference we can find is that the request content type header is set to "application/json; chartset=UTF-8" but in Chrome it's just "application/json". Going to fork and write a spec to see if this is actually the issue. Will post back results here.

from grape.

Why was this issue closed? I can't find any information about how to achieve posting JSON propertly to the Grape API

See this comment for more information

from grape.

I'm surprised this hasn't been addressed/fixed yet.

Here's how I hacked it in an old project:

# Overload grape to parse application/json objects until this is fixed
module Grape
  class API
    class << self

      def call(env)
        case env['CONTENT_TYPE']
        when /^application\/json/
          env.update('rack.request.form_hash' => JSON.parse(env['rack.input'].read), 'rack.request.form_input' => env['rack.input'])
        end
        logger.info "#{env['REQUEST_METHOD']} #{env['PATH_INFO']}"
        route_set.freeze.call(env)
      end

    end
  end
end

from grape.

@aaronchi thanks a bunch for that. So much cleaner than what I am doing now for every put or post.

I had to change a tiny thing the code provided by @aaronchi doesn't work for get and delete requests

# Overload grape to parse application/json objects until this is fixed
module Grape
  class API
    class << self

      def call(env)
        case env['CONTENT_TYPE']
        when /^application\/json/
          hash = env['rack.input'].read
          parsedForm = JSON.parse(hash) unless hash.blank?
          env.update('rack.request.form_hash' => parsedForm, 'rack.request.form_input' => env['rack.input']) if parsedForm
        end
        logger.info "#{env['REQUEST_METHOD']} #{env['PATH_INFO']}"
        route_set.freeze.call(env)
      end

    end
  end
end

Having to go through this makes me sick. But works for now, something like this should probably be implemented in the head branch but I am unsure about what to test for etc. Some guidance would be nice.

from grape.