--- parser3/src/targets/cgi/parser3.C 2020/11/16 16:15:36 1.316 +++ parser3/src/targets/cgi/parser3.C 2020/12/07 23:30:16 1.324 @@ -5,7 +5,7 @@ Author: Alexandr Petrosian (http://paf.design.ru) */ -volatile const char * IDENT_PARSER3_C="$Id: parser3.C,v 1.316 2020/11/16 16:15:36 moko Exp $"; +volatile const char * IDENT_PARSER3_C="$Id: parser3.C,v 1.324 2020/12/07 23:30:16 moko Exp $"; #include "pa_config_includes.h" @@ -40,16 +40,17 @@ volatile const char * IDENT_PARSER3_C="$ static const char* filespec_to_process = 0; // [file] static const char* httpd_host_port = 0; // -p option -static const char* config_filespec = 0; // -f option or from env or next to the executable +static const char* config_filespec = 0; // -f option or from env or next to the executable if exists static bool mail_received = false; // -m option? [asked to parse incoming message to $mail:received] -static char** argv_all = NULL; +static char* parser3_filespec = 0; // argv[0] +static char** argv_extra = NULL; // for signal handlers Request *request=0; -// for die error logging -static Request_info request_info; -static const char* filespec_4log = 0; // never null +// for error logging +static Request_info request_info; // global for correct log() reporting +static const char* filespec_4log = NULL; // null only if system-wide auto.p used // SAPI @@ -122,7 +123,7 @@ static void log(const char* fmt, ...) { } #endif -// appends to parser3.log located next to the executable if openable, to stderr otherwize +// appends to parser3.log located next to the config file if openable, to stderr otherwize void SAPI::log(SAPI_Info&, const char* fmt, ...) { va_list args; va_start(args,fmt); @@ -217,6 +218,31 @@ static void SIGPIPE_handler(int /*sig*/) } #endif +// requires pa_thread_request() in entry_exists() under Windows +static const char *locate_config(const char *config_filespec_option, const char *executable_path){ + filespec_4log=config_filespec_option; + if(!filespec_4log) + filespec_4log=getenv(PARSER_CONFIG_ENV_NAME); + if(!filespec_4log) + filespec_4log=getenv(REDIRECT_PREFIX PARSER_CONFIG_ENV_NAME); + if(!filespec_4log){ + // next to the executable + char *beside_executable_path = pa_strdup(executable_path); + bool stripped_filename = rsplit(beside_executable_path, '/') || rsplit(beside_executable_path, '\\'); + filespec_4log = pa_strcat(stripped_filename ? beside_executable_path : "." /* no path, just filename */ , "/" AUTO_FILE_NAME); + if(entry_exists(filespec_4log)) + return filespec_4log; +#ifdef SYSTEM_CONFIG_FILE + if(entry_exists(SYSTEM_CONFIG_FILE)){ + filespec_4log=NULL; + return SYSTEM_CONFIG_FILE; + } +#endif + return NULL; + } + return filespec_4log; +} + #ifdef WIN32 const char* maybe_reconstruct_IIS_status_in_qs(const char* original) { // 404;http://servername/page[?param=value...] @@ -266,40 +292,62 @@ public: } }; -static void connection_handler(SAPI_Info_HTTPD &info, HTTPD_Connection &connection){ +static void config_handler(SAPI_Info &info) { + char document_root_buf[MAX_STRING]; + full_disk_path("", document_root_buf, sizeof(document_root_buf)); + + request_info.document_root = document_root_buf; + request_info.uri = ""; + request_info.argv = argv_extra; + + // prepare to process request + Request r(info, request_info, String::Language(String::L_HTML|String::L_OPTIMIZE_BIT)); + { + // initing ::request ptr for signal handlers + RequestController rc(&r); + // only once + config_filespec = locate_config(config_filespec, parser3_filespec); + // process main auto.p only + r.core(config_filespec, false, String::Empty); + // clearing ::request in RequestController desctructor to prevent signal handlers from accessing invalid memory + } +} + +static void connection_handler(SAPI_Info_HTTPD &info, HTTPD_Connection &connection) { connection.read_header(); info.populate_env(); - // connection request info, still global for correct log() reporting - memset(&request_info, 0, sizeof(request_info)); - char document_root_buf[MAX_STRING]; full_disk_path("", document_root_buf, sizeof(document_root_buf)); + + memset(&request_info, 0, sizeof(request_info)); request_info.document_root = document_root_buf; request_info.path_translated = filespec_to_process; request_info.method = connection.method(); request_info.query_string = connection.query(); request_info.uri = request_info.strip_absolute_uri(connection.uri()); request_info.content_type = connection.content_type(); - request_info.content_length = connection.content_length(); + request_info.content_length = (size_t)connection.content_length(); request_info.cookie = info.get_env("HTTP_COOKIE"); request_info.mail_received = false; - request_info.argv = argv_all; + request_info.argv = argv_extra; // prepare to process request - Request request(info, request_info, String::Language(String::L_HTML|String::L_OPTIMIZE_BIT)); + Request r(info, request_info, String::Language(String::L_HTML|String::L_OPTIMIZE_BIT)); { // initing ::request ptr for signal handlers - RequestController rc(&request); + RequestController rc(&r); // process the request - request.core(config_filespec, strcasecmp(request_info.method, "HEAD")==0, String("httpd-main")); + r.core(config_filespec, strcasecmp(request_info.method, "HEAD")==0, String("httpd-main")); // clearing ::request in RequestController desctructor to prevent signal handlers from accessing invalid memory } } -static void httpd_mode(){ +static void httpd_mode() { int sock = HTTPD_Server::bind(httpd_host_port); + config_handler(*sapiInfo); + while(1){ try { HTTPD_Connection connection; @@ -394,30 +442,23 @@ static void real_parser_handler(bool cgi } request_info.content_type = getenv("CONTENT_TYPE"); - request_info.content_length = pa_atoul(getenv("CONTENT_LENGTH")); + request_info.content_length = (size_t)pa_atoul(getenv("CONTENT_LENGTH")); request_info.cookie = getenv("HTTP_COOKIE"); request_info.mail_received = mail_received; - request_info.argv = argv_all; + request_info.argv = argv_extra; #ifdef PA_DEBUG_CGI_ENTRY_EXIT - log("request_info: method=%s, uri=%s, q=%s, dr=%s, pt=%s, cookies=%s, cl=%u", - request_info.method, - request_info.uri, - request_info.query_string, - request_info.document_root, - request_info.path_translated, - request_info.cookie, - request_info.content_length); + log("request_info: method=%s, uri=%s, q=%s, dr=%s, pt=%s", request_info.method, request_info.uri, request_info.query_string, request_info.document_root, request_info.path_translated); #endif // prepare to process request - Request request(*sapiInfo, request_info, cgi ? String::Language(String::L_HTML|String::L_OPTIMIZE_BIT) : String::L_AS_IS); + Request r(*sapiInfo, request_info, cgi ? String::Language(String::L_HTML|String::L_OPTIMIZE_BIT) : String::L_AS_IS); { // initing ::request ptr for signal handlers - RequestController rc(&request); + RequestController rc(&r); // process the request - request.core(config_filespec, strcasecmp(request_info.method, "HEAD")==0); + r.core(locate_config(config_filespec, parser3_filespec), strcasecmp(request_info.method, "HEAD")==0); // clearing ::request in RequestController destructor to prevent signal handlers from accessing invalid memory } @@ -480,29 +521,13 @@ static void usage(const char* program) { exit(EINVAL); } -static const char *locate_config(const char *config_filespec_option, const char *executable_path){ - if(!(filespec_4log = config_filespec_option)) { - filespec_4log=getenv(PARSER_CONFIG_ENV_NAME); - if(!filespec_4log) - filespec_4log=getenv(REDIRECT_PREFIX PARSER_CONFIG_ENV_NAME); - if(!filespec_4log){ - // next to the executable - char *beside_executable_path = pa_strdup(executable_path); - bool stripped_filename = rsplit(beside_executable_path, '/') || rsplit(beside_executable_path, '\\'); - filespec_4log = pa_strcat(stripped_filename ? beside_executable_path : "." /* no path, just filename */ , "/" AUTO_FILE_NAME); - if(!entry_exists(filespec_4log)) - return NULL; - } - } - return filespec_4log; -} - int main(int argc, char *argv[]) { #ifdef PA_DEBUG_CGI_ENTRY_EXIT log("main: entry"); #endif + parser3_filespec = argv[0]; umask(2); // were we started as CGI? @@ -521,7 +546,7 @@ int main(int argc, char *argv[]) { char *raw_filespec_to_process = NULL; if(cgi) { raw_filespec_to_process=getenv("PATH_TRANSLATED"); - argv_all=argv + 1; + argv_extra=argv + 1; } else { int optind=1; while(optind < argc){ @@ -575,7 +600,7 @@ int main(int argc, char *argv[]) { usage(argv[0]); } - argv_all=argv + optind; + argv_extra=argv + optind; } #ifdef _MSC_VER @@ -598,15 +623,13 @@ int main(int argc, char *argv[]) { _CrtSetReportFile( _CRT_WARN, _CRTDBG_FILE_STDERR ); #endif - char filespec_to_process_buf[MAX_STRING]; - if(raw_filespec_to_process && *raw_filespec_to_process){ - full_disk_path(raw_filespec_to_process, filespec_to_process_buf, sizeof(filespec_to_process_buf)); - filespec_to_process=filespec_to_process_buf; - } - - config_filespec = locate_config(config_filespec, argv[0]); - try { // global try + char filespec_to_process_buf[MAX_STRING]; + if(raw_filespec_to_process && *raw_filespec_to_process){ + full_disk_path(raw_filespec_to_process, filespec_to_process_buf, sizeof(filespec_to_process_buf)); + filespec_to_process=filespec_to_process_buf; + } + REAL_PARSER_HANDLER(cgi); } catch(const Exception& e) { // exception in unhandled exception SAPI::die("%s", e.comment());