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
Added HTTP code 423 Locked #112
Conversation
@@ -32,6 +32,7 @@ Lead Maintainer: [Adam Bretz](https://github.com/arb) | |||
- [`Boom.rangeNotSatisfiable([message], [data])`](#boomrangenotsatisfiablemessage-data) | |||
- [`Boom.expectationFailed([message], [data])`](#boomexpectationfailedmessage-data) | |||
- [`Boom.badData([message], [data])`](#boombaddatamessage-data) | |||
- [`Boom.locked([message], [data])`](#boomlockedmessage-data) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove this; the TOC is auto generated on version bumps.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good to know.
- `data` - optional additional error data. | ||
|
||
```js | ||
Boom.locked('this account has been locked and it is your fault'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the it is your fault
is a bit harsh 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed. Not sure "account" is strictly necessary either. Why not just:
- The resource that is being accessed is locked
- This resource has been locked
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds good! (I know this is just an example but people look at this as an example so...)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Haha, fair enough :P
This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions. |
In response to #111, this adds
boom.locked()
which returns the 423 Locked response.