Support custom_payload for Rails 5 API's ActionController::API

[f3ndot]

Per the Rails 5 guide, if an app is using the API-only mode then they are to have ApplicationController inherit from ActionController::API instead of ActionController::Base. This breaks Lograge's custom payload feature.

My PR allows the developer to set the base class so that they can switch it to ActionController::API or something else should their app be very specialized.

[benlovell]

Awesome. Thanks!

[benlovell]

I've just cut a release for 0.7.0.

[f3ndot]

@benlovell oh no! I've made a critical error.

By specifying the constant ActionController::API in the config, it causes the JSON APIs to not spit out their payload:

Rails.application.configure do
  config.lograge.base_controller_class = ActionController::API
end

$ curl http://localhost:3000/
%

The fix is to specify the string of the class then constantize it in Lograge. I'll make a PR posthaste!

Rails.application.configure do
  config.lograge.base_controller_class = 'ActionController::API'
end

$ curl http://localhost:3000/
{"foo": "bar"}%

[benlovell]

Gah, no problem. Thanks!

[minhdanh]

I'm using both ActionController:Base and ActionController:API for my rails app. So how can I configure the custom payload feature for both of these controllers?

[f3ndot]

I suspect the code would have to change so base_controller_class can optionally be an array and the setup method iterates the members.