LearnersGuild / idm

identity management service
MIT License
2 stars 24 forks source link

Error handling improvements #102

Closed heyheyjp closed 8 years ago

heyheyjp commented 8 years ago

Fixes https://github.com/LearnersGuild/idm/issues/100.

heyheyjp commented 8 years ago

@jeffreywescott: would be awesome if you could give this some eyes, let me know what you think. Not super urgent, but I think it'll be important to pull these changes (in whatever their final state is) in before going live.

jeffreywescott commented 8 years ago

Love this. Made a few comments, none of which necessarily need to block the merge, but I thought I'd give you a chance to at least see them before merging. Definitely wondering about the MongoError and the best way to match DB errors.

heyheyjp commented 8 years ago

@jeffreywescott @bundacia: any outstanding concerns?