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

Fix: #75. Reference representation data, not message body. #76

Open
wants to merge 2 commits into
base: gh-pages
Choose a base branch
from

Conversation

ioggstream
Copy link
Contributor

This PR

Fixes the ambiguity about message body. In HTTP the message body
is subject to Transfer-Encoding and other transformations that might
invalidate a signature.

Digest for example, is calculated on the instance (rfc3230) now standardized as representation data (rfc7231).

The now deprecated, rfc723x-inconsistent Content-MD5 instead was calculated on the payload body.

index.xml Outdated
@@ -575,7 +575,7 @@ Note that the use of the `Digest` header field is per
<xref target="RFC3230">RFC 3230</xref>, <eref
target="https://tools.ietf.org/html/rfc3230#section-4.3.2">Section 4.3.2</eref>
and is included merely as a demonstration of how an implementer could include
information about the body of the message in the signature.
informations about the representation of the resource in the signature.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"information" has no plural form, so no "s" on the end.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done.

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.

2 participants