Provide handled: true attribute for Rails.error.report method, because it is required in Rails 7.0.

[skatkov]

handled: true attribute is required for Rails.error.report method in Rails 7.0, but not required in Rails 7.1.

See relevant code in rails.

• Rails 7.0 code, that require "handled: true" - https://github.com/rails/rails/blob/v7.0.8.4/activesupport/lib/active_support/error_reporter.rb#L95

• Rails 7.1 code, that doesn't require this attribute - https://github.com/rails/rails/blob/main/activesupport/lib/active_support/error_reporter.rb#L210

[julik]

If we have handled we know that it is unhandled if we expose it to the webhook sender in a way. Maybe use the value returned by the handler for it?

[skatkov]

@julik I don't think handled means what you think it means.

handled: true stands for error that was rescued and something was doneto deal with it. Since we're always reporting from a rescue block, it appears that we "handled" reported error.

Please check out the source of rails itself
https://github.com/rails/rails/blob/v7.0.8.4/activesupport/lib/active_support/error_reporter.rb#L57-L69

.record method re-raises errors, but they use handled: false for that.
.handle method will swallow an error and continue as nothing happened, and they use handled: true for that.