Changeset 1873 for cherokee/branches

Timestamp:

08/19/08 14:13:04 (5 months ago)

Author:

alo

Message:

--

Files:

• cherokee/branches/0.8 (modified) (1 prop)
• cherokee/branches/0.8/ChangeLog (modified) (1 diff)
• cherokee/branches/0.8/cherokee/connection-protected.h (modified) (1 diff)
• cherokee/branches/0.8/cherokee/connection.c (modified) (1 diff)
• cherokee/branches/0.8/cherokee/handler_dirlist.c (modified) (2 diffs)
• cherokee/branches/0.8/cherokee/rule_directory.c (modified) (1 diff)

cherokee/branches/0.8/ChangeLog

@@ -2 +2 @@
-
-        * 0.8.1 released
+
+        * cherokee/connection.c (cherokee_connection_set_redirect),
+        cherokee/connection-protected.h, cherokee/rule_directory.c,
+        cherokee/handler_dirlist.c, cherokee/connection.c: Fixes another
+        problem with redirections. It was failing to redirect requests
+        targeting a directory rule entry without a empty request (not even
+        the root slash). There is a new function now to perform this sort
+        of redirections, so it should be much safer to use it from
+        handler_dirilst and rule_directory.
-
-        * cherokee/handler_dirlist.c: Fixes a problem with redirections of

cherokee/branches/0.8/cherokee/connection-protected.h

@@ -241 +241 @@
-ret_t cherokee_connection_build_local_directory_userdir (cherokee_connection_t *conn, cherokee_virtual_server_t *vsrv, cherokee_config_entry_t *entry);
-ret_t cherokee_connection_clean_error_headers    (cherokee_connection_t *conn);
+ret_t cherokee_connection_set_redirect           (cherokee_connection_t *conn, cherokee_buffer_t *address);
-
-ret_t cherokee_connection_clean_for_respin       (cherokee_connection_t *conn);

cherokee/branches/0.8/cherokee/connection.c

@@ -2061 +2061 @@
-}
-#endif
+
+
+ret_t
+cherokee_connection_set_redirect (cherokee_connection_t *conn, cherokee_buffer_t *address)
+{
+        cuint_t len;
+
+        /* Build the redirection address
+         */
+        cherokee_buffer_clean (&conn->redirect);
+
+        len = conn->web_directory.len + 
+                address->len + 
+                conn->userdir.len + 
+                conn->host.len +
+                sizeof(":65535") +
+                sizeof("https://") + 4;
+        
+        cherokee_buffer_ensure_size (&conn->redirect, len);
+
+        /* In case the connection has a custom Document Root directory,
+         * it must add the web equivalent directory to the path (web_directory).
+         */
+        if (cherokee_connection_use_webdir (conn)) {
+                cherokee_buffer_add_buffer (&conn->redirect, &conn->web_directory);
+        }
+                
+        if (! cherokee_buffer_is_empty (&conn->host)) {
+                if (conn->socket.is_tls == TLS)
+                        cherokee_buffer_add_str (&conn->redirect, "https://");
+                else
+                        cherokee_buffer_add_str (&conn->redirect, "http://");
+                
+                cherokee_buffer_add_buffer (&conn->redirect, &conn->host);
+                
+                if (CONN_SRV(conn)->port != 80) {
+                        cherokee_buffer_add_str (&conn->redirect, ":");
+                        cherokee_buffer_add_long10 (&conn->redirect, CONN_SRV(conn)->port);
+                }
+        }
+        
+        if (! cherokee_buffer_is_empty (&conn->userdir)) {
+                cherokee_buffer_add_str (&conn->redirect, "/~");
+                cherokee_buffer_add_buffer (&conn->redirect, &conn->userdir);
+        }
+        
+        cherokee_buffer_add_buffer (&conn->redirect, address);
+        return ret_ok;
+}

cherokee/branches/0.8/cherokee/handler_dirlist.c

@@ -453 +453 @@
-check_request_finish_with_slash (cherokee_handler_dirlist_t *dhdl)
-{
-        cuint_t                len;
-        cherokee_connection_t *conn = HANDLER_CONN(dhdl);
-
@@ -459 +458 @@
-            (!cherokee_buffer_is_ending (&conn->request, '/'))) 
-        {
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
+
+
+
-                
-                conn->error_code = http_moved_permanently;

cherokee/branches/0.8/cherokee/rule_directory.c

@@ -69 +69 @@
-            (cherokee_buffer_cmp_buf (&conn->request, &rule->directory) == 0))
-        {
-ensure_size (&conn->redirect, conn->request.len + 4
-buffer_add_buffer (&conn->redirect
-add (&conn->redirect, "/"
+add_str (&conn->request, "/"
+connection_set_redirect (conn
+drop_ending (&conn->request
-
-                TRACE(ENTRIES, "Had to redirect to: %s\n", conn->redirect.buf);