Make the Http1Parser::request_offets details private

diff --git a/src/client_side.cc b/src/client_side.cc
index f937d20f1118ccdf0b79080b2eb4765c1aacbcd0..dea06e9581ff94ed6009b474e9bf3bcfa5c46371 100644 (file)
--- a/src/client_side.cc
+++ b/src/client_side.cc
@@ -2346,7 +2346,10 @@ parseHttpRequest(ConnStateData *csd, const Http::Http1ParserPointer &hp)
 
     // XXX: crop this dump at the end of headers. No need for extras
     debugs(11, 2, "HTTP Client " << csd->clientConnection);
-    debugs(11, 2, "HTTP Client REQUEST:\n---------\n" << (hp->buf) + hp->req.m_start << "\n----------");
+    debugs(11, 2, "HTTP Client REQUEST:\n---------\n" <<
+           hp->method() << " " << hp->requestUri() << " " << hp->messageProtocol() << "\n" <<
+           hp->rawHeaderBuf() <<
+           "\n----------");
 
     result->flags.parsed_ok = 1;
     return result;

diff --git a/src/http/Http1Parser.h b/src/http/Http1Parser.h
index f981e1049b9fa5d55000ffd9b60ba6b8e5d99727..bee6de298d862ecf4607899129c0f2e13a2f62c8 100644 (file)
--- a/src/http/Http1Parser.h
+++ b/src/http/Http1Parser.h
@@ -83,14 +83,6 @@ public:
     const char *buf;
     int bufsiz;
 
-    /// Offsets for pieces of the (HTTP request) Request-Line as per RFC 2616
-    struct request_offsets {
-        int start, end;
-        int m_start, m_end; // method
-        int u_start, u_end; // url
-        int v_start, v_end; // version (full text)
-    } req;
-
     /// the protocol label for this message
     const AnyP::ProtocolVersion & messageProtocol() const {return msgProtocol_;}
 
@@ -111,6 +103,14 @@ private:
     bool skipGarbageLines();
     int parseRequestFirstLine();
 
+    /// Offsets for pieces of the (HTTP request) Request-Line as per RFC 2616
+    struct request_offsets {
+        int start, end;
+        int m_start, m_end; // method
+        int u_start, u_end; // url
+        int v_start, v_end; // version (full text)
+    } req;
+
     /// byte offset for non-parsed region of the buffer
     size_t parseOffset_;