flewkey/cuberite-2a
1
0
Fork 0
Code Releases Activity

Authentication should work now..

Browse Source 
git-svn-id: http://mc-server.googlecode.com/svn/trunk@654 0a769ca7-a7f5-676a-18bf-c427514a06d6
This commit is contained in:
lapayo94@gmail.com 2012-07-10 07:16:42 +00:00
parent 38459c9c17
commit 7276cc0f9b
1 changed files with 8 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
source/cAuthenticator.cpp
View File

@ -163,10 +163,10 @@ bool cAuthenticator::AuthFromAddress(const AString & a_Server, const AString & a
Link.SendMessage( AString( "GET " + a_Address + " HTTP/1.1\r\n" ).c_str()); Link.SendMessage( AString( "GET " + a_Address + " HTTP/1.1\r\n" ).c_str());
Link.SendMessage( AString( "User-Agent: MCServer\r\n" ).c_str()); Link.SendMessage( AString( "User-Agent: MCServer\r\n" ).c_str());
// FIXME: AString( "Host: %s\r\n", a_Server.c_str() ).c_str() doesn't work. Causes segfault. Link.SendMessage( AString( "Host: " + a_Server + "\r\n" ).c_str());
Link.SendMessage( AString( "Host: session.minecraft.net\r\n" ).c_str()); //Link.SendMessage( AString( "Host: session.minecraft.net\r\n" ).c_str());
Link.SendMessage( AString( "Accept: */*\r\n" ).c_str()); Link.SendMessage( AString( "Accept: */*\r\n" ).c_str());
Link.SendMessage( AString( "Connection: keep-alive\r\n" ).c_str()); Link.SendMessage( AString( "Connection: close\r\n" ).c_str()); //Close so we don´t have to mess with the Content-Length :)
Link.SendMessage( AString( "\r\n" ).c_str()); Link.SendMessage( AString( "\r\n" ).c_str());
AString DataRecvd; AString DataRecvd;
Link.ReceiveData(DataRecvd); Link.ReceiveData(DataRecvd);
@ -266,14 +266,15 @@ bool cAuthenticator::AuthFromAddress(const AString & a_Server, const AString & a
std::string Result; std::string Result;
ss >> Result; ss >> Result;
LOGINFO("Got result: %s", Result.c_str()); LOGINFO("Got result: %s", Result.c_str());
// if (Result.compare("YES") == 0) //if (Result.compare("3") == 0) // FIXME: Quick and dirty hack to support auth
// TODO: I'm assuming the only response containing 3 letters is "YES", but proper reading of the return code //Lapayo: Wtf 3?
// would be preferred. if (Result.compare("YES") == 0) //Works well
if (Result.compare("3") == 0) // FIXME: Quick and dirty hack to support auth
{ {
LOGINFO("Result was \"YES\", so player is authenticated!"); LOGINFO("Result was \"YES\", so player is authenticated!");
return true; return true;
} }
LOGINFO("Result was \"%s\", so player is NOT authenticated!", Result.c_str()); LOGINFO("Result was \"%s\", so player is NOT authenticated!", Result.c_str());
return false; return false;
} }