Remove unnecessary dump_header functionality

This functionality is duplicate of the same idea in HTTPCommands.cc;
however, in HTTPCommands.cc, the handler is already hooked up to the
configuration file log levels.

Author: bbockelm

src/CurlUtil.cc

@@ -46,54 +46,24 @@ HandlerQueue::HandlerQueue() {
 	m_write_fd = filedes[1];
 };
 
-namespace {
-
-// Simple debug function for getting information from libcurl; to enable, you
-// need to recompile with GetHandle(true);
-int dump_header(CURL *handle, curl_infotype type, char *data, size_t size,
-				void *clientp) {
-	(void)handle;
-	(void)clientp;
-
-	switch (type) {
-	case CURLINFO_HEADER_OUT:
-		printf("Header > %s\n", std::string(data, size).c_str());
-		break;
-	default:
-		printf("Info: %s", std::string(data, size).c_str());
-		break;
+CURL *HandlerQueue::GetHandle() {
+	if (!m_handles.empty()) {
+		CURL *result = m_handles.top();
+		m_handles.pop();
+		return result;
 	}
-	return 0;
-}
 
-} // namespace
-
-CURL *GetHandle(bool verbose) {
 	auto result = curl_easy_init();
 	if (result == nullptr) {
 		return result;
 	}
 
 	curl_easy_setopt(result, CURLOPT_USERAGENT, "xrootd-s3/devel");
-	curl_easy_setopt(result, CURLOPT_DEBUGFUNCTION, dump_header);
-	if (verbose)
-		curl_easy_setopt(result, CURLOPT_VERBOSE, 1L);
-
 	curl_easy_setopt(result, CURLOPT_BUFFERSIZE, 32 * 1024);
 
 	return result;
 }
 
-CURL *HandlerQueue::GetHandle() {
-	if (!m_handles.empty()) {
-		CURL *result = m_handles.top();
-		m_handles.pop();
-		return result;
-	}
-
-	return ::GetHandle(false);
-}
-
 void HandlerQueue::RecycleHandle(CURL *curl) { m_handles.push(curl); }
 
 void HandlerQueue::Produce(HTTPRequest *handler) {
@@ -233,7 +203,7 @@ void CurlWorker::Run() {
 
 			auto curl = queue.GetHandle();
 			if (curl == nullptr) {
-				m_logger.Log(LogMask::Debug, "Run",
+				m_logger.Log(LogMask::Warning, "Run",
 							 "Unable to allocate a curl handle");
 				op->Fail("E_NOMEM", "Unable to get allocate a curl handle");
 				continue;