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

Provide more specific error handling. #18

Open
LATaylor-guardian opened this issue Feb 12, 2016 · 0 comments
Open

Provide more specific error handling. #18

LATaylor-guardian opened this issue Feb 12, 2016 · 0 comments

Comments

@LATaylor-guardian
Copy link

Example: Cancel reindex

There are three error outcomes here:

  1. The request to cancel the reindex from the content source tool fails and therefore this is an internal server error.
  2. No content source exists for the given content source and therefore this is a bad request.
    3.There is no running reindex for this content source and therefore this is a bad request.

I think the CustomError type impedes here. Maybe I can group those specific errors under a new type, CancelReindexError and match on that to resolve the error response more specifically.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant