Skip to content

Enhancement: Centralize call to handle_error() #20

Description

@lifeBCE
erics-mbp:CloudMux ericthompson$ grep -nr handle_error app/* | wc -l
 339

There are currently 339 calls to handle_error() throughout all resource APIs. We can improve this significantly by removing potentially a couple thousand lines of code from all resource APIs that define their own exception handling block and make a call to handle_error() and by adding a global error handler to ResourceApiBase.

error do
    handle_error(env['sinatra.error'])
end

This method would be executed any time an exception is raised from a route block or a filter. This means we should be able to remove all exception handlers (begin...rescue) from all resource APIs and allow any exception to bubble up to error() in ResourceApiBase.

http://www.sinatrarb.com/intro.html#Error

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Fields

    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions