Fix empty command bug

This fixes a bug that causes gitano to crash when it's given an empty
command list

Opening and closing of the log should be done in the application,
so it can log before checking for authorization, log after, and
clean up at the end

1 files changed, 9 insertions, 1 deletions

diff --git a/bin/gitano-smart-http.cgi.in b/bin/gitano-smart-http.cgi.in
index f294b28..6f8fdee 100755
--- a/bin/gitano-smart-http.cgi.in
+++ b/bin/gitano-smart-http.cgi.in
@@ -59,8 +59,14 @@ if request_method == "GET" or request_method == "POST" then
    local user = os.getenv("REMOTE_USER") or "gitano/anonymous"
    local cmdline = parse_request(request_method)
 
+   local start_log_level = gitano.log.get_level()
+   gitano.log.cap_level(gitano.log.level.INFO)
+   local transactionid = gitano.log.syslog.open()
+
    if cmdline and gitano.auth.is_authorized(user, "http", cmdline,
-                                            os.getenv("GITANO_ROOT")) then
+                                            os.getenv("GITANO_ROOT"),
+                                            transactionid,
+                                            start_log_level) then
       local proc = subprocess.spawn_simple({"git", "http-backend"})
       local exit_code
 
@@ -72,6 +78,8 @@ if request_method == "GET" or request_method == "POST" then
    else
       stdout:write("Status: 403 Forbidden\r\n\r\n")
    end
+
+   gitano.log.syslog.close()
 else
    stdout:write("Status: 405 Method Not Allowed\r\n")
    stdout:write("Allow: GET, POST\r\n\r\n")