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

Precondition failed #66

Merged
merged 2 commits into from
Aug 11, 2016
Merged

Precondition failed #66

merged 2 commits into from
Aug 11, 2016

Conversation

no-reply
Copy link

No description provided.

@coveralls
Copy link

coveralls commented Aug 10, 2016

Coverage Status

Changes Unknown when pulling 4416f03 on precondition-failed into * on master*.

@jcoyne
Copy link
Member

jcoyne commented Aug 11, 2016

Needs a rebase

Tom Johnson added 2 commits August 10, 2016 17:10
This is a more appropriate name for the error message. HTTP 412 status
codes can arise for precondition failures other than etag issues, and
not all etag issues are correctly characterized as mismatches.

Closes #63.
Reintroduces `Ldp::ETagMismatch` as an alias to `PreconditionFailed` for
backwards compatibility.

Revert this to end support for the old name.
@no-reply no-reply force-pushed the precondition-failed branch from 4416f03 to 40dfa63 Compare August 11, 2016 00:10
@no-reply
Copy link
Author

@coveralls
Copy link

coveralls commented Aug 11, 2016

Coverage Status

Coverage increased (+0.03%) to 82.077% when pulling 40dfa63 on precondition-failed into 5c5ec56 on master.

@jcoyne jcoyne merged commit 3d9302f into master Aug 11, 2016
@jcoyne jcoyne deleted the precondition-failed branch August 11, 2016 00:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants