Fix a bug in error parsing #21

Merged
joeybaker merged 3 commits from better-error into master 2016-09-05 19:25:14 +00:00
joeybaker commented 2016-09-05 00:29:39 +00:00 (Migrated from github.com)

Fixes a case where the server could hang b/c we'd never call reply. Also adds some better parsing around PG errors.

This is a patch release.

Fixes a case where the server could hang b/c we'd never call `reply`. Also adds some better parsing around PG errors. This is a patch release.
joeybaker commented 2016-09-05 00:39:45 +00:00 (Migrated from github.com)

@mdibaiee this is the kind of thing I'd just merge myself, but I figure it's better to let you see it since you've gotta push the npm publish button.

@mdibaiee this is the kind of thing I'd just merge myself, but I figure it's better to let you see it since you've gotta push the `npm publish` button.
mdibaiee commented 2016-09-05 03:15:59 +00:00 (Migrated from github.com)

You're right, it seems it's possible to have collaborators in npm, too, I didn't know that. Please feel free to merge these kind of things yourself and publish.

I just added you as collaborator: https://www.npmjs.com/package/hapi-sequelize-crud

You're right, it seems it's possible to have collaborators in npm, too, I didn't know that. Please feel free to merge these kind of things yourself and publish. I just added you as collaborator: https://www.npmjs.com/package/hapi-sequelize-crud
joeybaker commented 2016-09-05 19:31:03 +00:00 (Migrated from github.com)

Published as 2.5.4

Published as 2.5.4
joeybaker commented 2016-09-05 19:31:10 +00:00 (Migrated from github.com)

Thanks @mdibaiee!

Thanks @mdibaiee!
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: thereadme/hapi-sequelize-crud#21
No description provided.