Merge pull request ClickHouse#80709 from ClickHouse/chesema-debug-tttp-responce

add more debug when http message parsing fails

Author: CheSema

base/poco/Net/src/HTTPResponse.cpp

@@ -108,23 +108,23 @@ HTTPResponse::HTTPResponse():
 {
 }
 
-	
+
 HTTPResponse::HTTPResponse(HTTPStatus status, const std::string& reason):
 	_status(status),
 	_reason(reason)
 {
 }
 
 
-	
+
 HTTPResponse::HTTPResponse(const std::string& version, HTTPStatus status, const std::string& reason):
 	HTTPMessage(version),
 	_status(status),
 	_reason(reason)
 {
 }
 
-	
+
 HTTPResponse::HTTPResponse(HTTPStatus status):
 	_status(status),
 	_reason(getReasonForStatus(status))
@@ -155,8 +155,8 @@ void HTTPResponse::setStatus(const std::string& status)
 {
 	setStatus((HTTPStatus) NumberParser::parse(status));
 }
-	
-	
+
+
 void HTTPResponse::setReason(const std::string& reason)
 {
 	_reason = reason;
@@ -169,7 +169,7 @@ void HTTPResponse::setStatusAndReason(HTTPStatus status, const std::string& reas
 	_reason = reason;
 }
 
-	
+
 void HTTPResponse::setStatusAndReason(HTTPStatus status)
 {
 	setStatusAndReason(status, getReasonForStatus(status));
@@ -181,7 +181,7 @@ void HTTPResponse::setDate(const Poco::Timestamp& dateTime)
 	set(DATE, DateTimeFormatter::format(dateTime, DateTimeFormat::HTTP_FORMAT));
 }
 
-	
+
 Poco::Timestamp HTTPResponse::getDate() const
 {
 	const std::string& dateTime = get(DATE);
@@ -240,7 +240,7 @@ void HTTPResponse::read(std::istream& istr)
 	std::string version;
 	std::string status;
 	std::string reason;
-	
+
 	int ch =  istr.get();
 	if (istr.bad()) throw NetException("Error reading HTTP response header");
 	if (ch == eof) throw NoMessageException();
@@ -260,9 +260,17 @@ void HTTPResponse::read(std::istream& istr)
 	HTTPMessage::read(istr);
 	ch = istr.get();
 	while (ch != '\n' && ch != eof) { ch = istr.get(); }
-	setVersion(version);
-	setStatus(status);
-	setReason(reason);
+
+	try
+	{
+		setVersion(version);
+		setStatus(status);
+		setReason(reason);
+	}
+	catch (SyntaxException & e)
+	{
+		throw SyntaxException(e.message(), ", while reading HTTP response: version='" + version + "', status='" + status + "', reason='" + reason + "'");
+	}
 }
 
 
@@ -282,7 +290,7 @@ const std::string& HTTPResponse::getReasonForStatus(HTTPStatus status)
 		return HTTP_REASON_CREATED;
 	case HTTP_ACCEPTED:
 		return HTTP_REASON_ACCEPTED;
-	case HTTP_NONAUTHORITATIVE:	
+	case HTTP_NONAUTHORITATIVE:
 		return HTTP_REASON_NONAUTHORITATIVE;
 	case HTTP_NO_CONTENT:
 		return HTTP_REASON_NO_CONTENT;