hydrar/mistserver
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fixed HTTP parser to no longer require "HTTP*" as protocol when creating requests/responses.

Browse source
This commit is contained in:
Thulinma 2014-05-04 01:25:21 +02:00
parent 9587d2fff6
commit 0fa40bb373
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
lib/http_parser.cpp
View file

@ -33,7 +33,7 @@ void HTTP::Parser::Clean(){
std::string & HTTP::Parser::BuildRequest(){ std::string & HTTP::Parser::BuildRequest(){
/// \todo Include GET/POST variable parsing? /// \todo Include GET/POST variable parsing?
std::map<std::string, std::string>::iterator it; std::map<std::string, std::string>::iterator it;
if (protocol.size() < 5 || protocol.substr(0, 4) != "HTTP"){ if (protocol.size() < 5 || protocol[4] != '/'){
protocol = "HTTP/1.0"; protocol = "HTTP/1.0";
} }
builder = method + " " + url + " " + protocol + "\r\n"; builder = method + " " + url + " " + protocol + "\r\n";
@ -52,7 +52,7 @@ std::string & HTTP::Parser::BuildRequest(){
void HTTP::Parser::SendRequest(Socket::Connection & conn){ void HTTP::Parser::SendRequest(Socket::Connection & conn){
/// \todo Include GET/POST variable parsing? /// \todo Include GET/POST variable parsing?
std::map<std::string, std::string>::iterator it; std::map<std::string, std::string>::iterator it;
if (protocol.size() < 5 || protocol.substr(0, 4) != "HTTP"){ if (protocol.size() < 5 || protocol[4] != '/'){
protocol = "HTTP/1.0"; protocol = "HTTP/1.0";
} }
builder = method + " " + url + " " + protocol + "\r\n"; builder = method + " " + url + " " + protocol + "\r\n";
@ -76,7 +76,7 @@ void HTTP::Parser::SendRequest(Socket::Connection & conn){
std::string & HTTP::Parser::BuildResponse(std::string code, std::string message){ std::string & HTTP::Parser::BuildResponse(std::string code, std::string message){
/// \todo Include GET/POST variable parsing? /// \todo Include GET/POST variable parsing?
std::map<std::string, std::string>::iterator it; std::map<std::string, std::string>::iterator it;
if (protocol.size() < 5 || protocol.substr(0, 4) != "HTTP"){ if (protocol.size() < 5 || protocol[4] != '/'){
protocol = "HTTP/1.0"; protocol = "HTTP/1.0";
} }
builder = protocol + " " + code + " " + message + "\r\n"; builder = protocol + " " + code + " " + message + "\r\n";
@ -102,7 +102,7 @@ std::string & HTTP::Parser::BuildResponse(std::string code, std::string message)
void HTTP::Parser::SendResponse(std::string code, std::string message, Socket::Connection & conn){ void HTTP::Parser::SendResponse(std::string code, std::string message, Socket::Connection & conn){
/// \todo Include GET/POST variable parsing? /// \todo Include GET/POST variable parsing?
std::map<std::string, std::string>::iterator it; std::map<std::string, std::string>::iterator it;
if (protocol.size() < 5 || protocol.substr(0, 4) != "HTTP"){ if (protocol.size() < 5 || protocol[4] != '/'){
protocol = "HTTP/1.0"; protocol = "HTTP/1.0";
} }
builder = protocol + " " + code + " " + message + "\r\n"; builder = protocol + " " + code + " " + message + "\r\n";