Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #262: JSONErrorsMixin not returning correct results #263

Merged
merged 1 commit into from
Oct 31, 2017

Commits on Oct 18, 2017

  1. Fix jupyter-server#262: JSONErrorsMixin not returning correct results

    After Notebook 5.2.0 is installed, JSONErrorsMixin.write_error is
    used to convert errors into JSON responses.  Previously, Notebook's
    `@json_errors` decorator performed this action. Since the decorator's
    replacement, `APIHandler.write_error`, doesn't adequately set the
    `reason` field, other tests fail so its not sufficient to drop the
    use of JSONErrorsMixin.  As a result, JSONErrorsMixin now incorporates
    the appropriate code from `APIHandler.write_error` (in particular the
    setting of the `traceback` field) - in addition to appropriately
    setting the `reason` field.  This mixin should continue to be used
    in the handler class hierachy until `APIHandler` is fixed.
    
    I have tested these changes with Notebook 5.1.0 and 5.2.0 installed.
    kevin-bates committed Oct 18, 2017
    Configuration menu
    Copy the full SHA
    9896a01 View commit details
    Browse the repository at this point in the history