The HttpMsg::protocol removed with "Bug 1961: pt1: URL handling redesign" patch,
and as a result the eCAP squid subsystem does not build because used this memberto implement libecap::RequestLine and libecap::StatusLine classes.
The HttpMsg::protocol used to hold the protocol part of the request URI.
However the libecap::FirstLine::protocol() is meant for things like
* the HTTP-Version part of HTTP messages (in RFC 2616 terminology) or
* the ICAP-Version part of ICAP messages (in RFC 3507 terminology).
It is not related to the URI.
This patch fix this and now libecap::RequestLine and libecap::StatusLine
implemented to return the protocol information from request or status line
of headers.
Adaptation::Ecap::FirstLineRep::protocol() const
{
// TODO: optimize?
- switch (theMessage.protocol) {
+ switch (theMessage.http_ver.protocol) {
case AnyP::PROTO_HTTP:
return libecap::protocolHttp;
case AnyP::PROTO_HTTPS:
Adaptation::Ecap::FirstLineRep::protocol(const Name &p)
{
// TODO: what happens if we fail to translate some protocol?
- theMessage.protocol = TranslateProtocolId(p);
+ theMessage.http_ver.protocol = TranslateProtocolId(p);
}
AnyP::ProtocolType