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

Error handling #41

Open
2 of 7 tasks
SaschaAdler opened this issue Feb 18, 2021 · 2 comments
Open
2 of 7 tasks

Error handling #41

SaschaAdler opened this issue Feb 18, 2021 · 2 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@SaschaAdler
Copy link

SaschaAdler commented Feb 18, 2021

There are a lot of things in Kivik's codebase that can throw uncaught exceptions. These exceptions should be caught, logged, and the code should gracefully fail accordingly.

  • YAML.parse can't parse the RC file
  • <deployment> does not refer to a key in the RC file
  • Docker errors (src/Container.js)
  • require, fs.read*
  • nano can't communicate with couch
  • couch forbids a nano request
  • Things are stopped before they're started, deployed before they're loaded, etc.
@SaschaAdler SaschaAdler self-assigned this Feb 18, 2021
@SaschaAdler SaschaAdler added this to the v2.0.0-beta milestone Feb 19, 2021
@SaschaAdler SaschaAdler added the enhancement New feature or request label Feb 19, 2021
@SaschaAdler
Copy link
Author

I removed a couple of tasks because Kivik no longer invokes ajv directly. #50

@SaschaAdler
Copy link
Author

I'm confident other things are sorted out reasonably well, although I'll have to test them (#57)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant