Remove @Experimental for QUERY method and implement conformance to RFC 10008#673
Remove @Experimental for QUERY method and implement conformance to RFC 10008#673desiderantes wants to merge 4 commits into
Conversation
|
@desiderantes I will do my best to get around to reviewing this change-set and including it in the upcoming 5.5-beta2 release. Please note though we usually do not upgrade to RFCs piecemeal. We need to be sure that at the very least all hard requirements are met Please also add RFC 10008 to the list of RFCs the library conforms to in README.md |
Hi, in this case, the prior change implemented basic QUERY method support, while the RFC was in draft status. After it graduated from that status, there were now missing parts for conformance, which I add with this PR.
Done |
@desiderantes What I am trying to say we need to be sure the protocol code fully conforms to RFC 10008 in terms of hard requirements before we can remove |
Ah, got you, no problem. Yes, the required changes in this project were minimal from my point of view. There is another PR for the client itself that handles the changes required there, and I referenced this PR from that one. |
Removing Experimental Markings
Since the QUERY method draft has been promoted to a proper RFC, this change removes the @experimental tag from QUERY method.
RFC 10008 Conformance
According to Section 2 of RFC 10008 https://www.rfc-editor.org/rfc/rfc10008.html:
│ Servers MUST fail the request if the Content-Type request field is missing or is inconsistent with the request content.
I've added code to enforce this constraint.
This is follow up of #499