Browse Source

do not attempt to get the server status if no status URL is specified

diafero 14 years ago
parent
commit
5e3223b542
  1. 2
      Sources/Plasma/Apps/plClient/winmain.cpp
  2. 27
      Sources/Plasma/Apps/plUruLauncher/Main.cpp
  3. 6
      Sources/Plasma/NucleusLib/pnNetBase/Private/pnNbSrvs.cpp

2
Sources/Plasma/Apps/plClient/winmain.cpp

@ -1193,7 +1193,7 @@ void StatusCallback(void *param)
curl_easy_setopt(hCurl, CURLOPT_WRITEFUNCTION, &CurlCallback);
curl_easy_setopt(hCurl, CURLOPT_WRITEDATA, param);
if (curl_easy_perform(hCurl) != 0)
if (statusUrl[0] && curl_easy_perform(hCurl) != 0) // only perform request if there's actually a URL set
PostMessage(hwnd, WM_USER_SETSTATUSMSG, 0, (LPARAM) curlError);
for(unsigned i = 0; i < UPDATE_STATUSMSG_SECONDS && s_loginDlgRunning; ++i)

27
Sources/Plasma/Apps/plUruLauncher/Main.cpp

@ -605,39 +605,19 @@ static size_t CurlCallback(void *buffer, size_t size, size_t nmemb, void *)
//============================================================================
static void StatusCallback(void *)
{
CURL *hCurl;
char *serverUrl = hsWStringToString(GetServerStatusUrl());
char *serverStatus = hsWStringToString(BuildTypeServerStatusPath());
char serverUrl[256];
snprintf(serverUrl, 256, "http://support.cyanworlds.com%s", serverStatus);
delete [] serverStatus;
hCurl = curl_easy_init();
CURL * hCurl = curl_easy_init();
curl_easy_setopt(hCurl, CURLOPT_ERRORBUFFER, s_curlError);
// update while we are running
while(!s_shutdown)
{
if(BuildTypeServerStatusPath())
{
//TODO: Get a server status path from the server.ini, without
// pulling in all the annoying pfConsole dependencies.
// Alternatively, make a better launcher...
curl_easy_setopt(hCurl, CURLOPT_USERAGENT, "UruClient/1.0");
curl_easy_setopt(hCurl, CURLOPT_URL, serverUrl);
if(StrLen(s_postKey))
{
char *safeData = curl_easy_escape(hCurl, s_postKey, strlen(s_postKey));
curl_easy_setopt(hCurl, CURLOPT_POST, 1);
curl_easy_setopt(hCurl, CURLOPT_POSTFIELDS, safeData);
curl_free(safeData);
}
if (curl_easy_perform(hCurl) != 0)
if (serverUrl[0] && curl_easy_perform(hCurl) != 0) // only perform request if there's actually a URL set
SetStatusText(s_curlError);
}
for(unsigned i = 0; i < UPDATE_STATUSMSG_SECONDS && !s_shutdown; ++i)
{
@ -646,6 +626,7 @@ static void StatusCallback(void *)
}
curl_easy_cleanup(hCurl);
delete [] serverUrl;
s_statusEvent.Signal();
}

6
Sources/Plasma/NucleusLib/pnNetBase/Private/pnNbSrvs.cpp

@ -137,9 +137,9 @@ void SetGateKeeperSrvHostname (const wchar addr[]) {
//============================================================================
// User-visible Server
//============================================================================
static wchar s_serverStatusUrl[256];
static wchar s_serverSignupUrl[256];
static wchar s_serverName[256];
static wchar s_serverStatusUrl[256] = {0};
static wchar s_serverSignupUrl[256] = {0};
static wchar s_serverName[256] = {0};
//============================================================================
const wchar *GetServerStatusUrl () {

Loading…
Cancel
Save