Missing ActionContoller event fields due to `#process_action` monkey patch

[avlazarov]

Hey!

I'm trying to get information about the request's headers after ActiveSupport::Notifications.subscribe 'process_action.action_controller', but the :headers key is missing from the event's payload. As far as I can see, this gem monkey patches ActionController::Instrumentation#process_action hence a few Rails generated fields will be missing: headers and request.

Could these extra fields be still visible for subscribers of the ActionController?

[shadabahmed]

@avlazarov good point - would you be able to create a PR ?

cc @petergoldstein

[avlazarov]

@shadabahmed Sure, but before that I have a few questions about the current guards, namely:

logstasher/lib/logstasher/rails_ext/action_controller/metal/instrumentation.rb

Line 11 in 9de62bc

format: request.format.try(:ref),

logstasher/lib/logstasher/rails_ext/action_controller/metal/instrumentation.rb

Lines 13 to 17 in 9de62bc

	path: begin
	request.fullpath
	rescue StandardError
	'unknown'
	end

I've gone through all versions that this gem supports (Rails 5.2+) and none seem to be defensive about format and path:

• 5.2
• 6-0
• 6-1
• 7-0

Am I missing something or can I just use the instrumentation logic from Rails 7.0, without any guards?