Re: [amqp-comment] amqp over http draft

[Gordon Sim <gsim@redhat.com>]

On 28/05/2019 2:42 pm, Clemens Vasters wrote:
> My alternate proposal would look like this, and turns "http:version" into a required "http:req" or "http:res" property.
>
> 4.1.1	Request Line
> The HTTP request-line contains the HTTP method, the request-target, and the HTTP version.
> *	HTTP method MUST be set in the properties section, subject field.
> *	HTTP request-target value MUST be URI-decoded and set in the properties section, to field.
> *	The version value MUST be set in application-properties section, as value of the âhttp:reqâ string property.
> 4.1.2	Status Line
> The HTTP status-line contains the HTTP version, the status code, and the reason phrase.
> *	The version value MUST be set in application-properties section, as value of the âhttp:rspâ string property.
> *	HTTP status code MUST be set in the properties section, subject field.
> *	HTTP reason phrase is OPTIONAL. The value is set in application-properties section, as value of the âhttp:reasonâ string property. The property SHOULD be omitted because RFC7230 3.1.2 RECOMMENDS it being ignored by clients.

That is better than the current draft in my opinion.