Changeset 6861

Timestamp:

Oct 7, 2008, 1:25:29 AM (13 years ago)

Author:

charles

Message:

(rpc) if libt can't find the Clutch files, give a helpful 404 message for end-users and binary packagers about how to use CLUTCH_HOME and PACKAGE_DATA_DIR.

Location:

trunk/libtransmission

Files:

• platform.c (modified) (1 diff)
• rpc-server.c (modified) (2 diffs)

trunk/libtransmission/platform.c

@@ -611 +611 @@
         }
 
-        if( !*path )
-        {
-            tr_strlcpy( path, "/dev/null", sizeof( path ) );
-            tr_err( _(
-                       "Couldn't find the web interface's files!  To customize this, set the CLUTCH_HOME environmental variable to the folder where index.html is located." ) );
-        }
-
         s = tr_strdup( path );
     }

trunk/libtransmission/rpc-server.c

@@ -71 +71 @@
     evbuffer_add_printf( body, "<h1>%d: %s</h1>", code, code_text );
     if( text )
-        evbuffer_add_printf( body, "<h2>%s</h2>", text );
+        evbuffer_add_printf( body, "%s", text );
     evhttp_send_reply( req, code, code_text, body );
     evbuffer_free( body );
@@ -321 +321 @@
                struct tr_rpc_server *  server )
 {
-    char * pch;
-    char * subpath;
-    char * filename;
     const char * clutchDir = tr_getClutchDir( server->session );
 
     assert( !strncmp( req->uri, "/transmission/web/", 18 ) );
 
-    subpath = tr_strdup( req->uri + 18 );
-    if(( pch = strchr( subpath, '?' )))
-        *pch = '\0';
-
-    filename = *subpath
-        ? tr_strdup_printf( "%s%s%s", clutchDir, TR_PATH_DELIMITER_STR, subpath )
-        : tr_strdup_printf( "%s%s%s", clutchDir, TR_PATH_DELIMITER_STR, "index.html" );
-
-    serve_file( req, filename );
-
-    tr_free( filename );
-    tr_free( subpath );
+    if( !clutchDir || !*clutchDir )
+    {
+        send_simple_response( req, HTTP_NOTFOUND,
+            "<p>Couldn't find Transmission's web interface files!</p>"
+            "<p>Users: to tell Transmission where to look, "
+            "set the TRANSMISSION_WEB_HOME environmental "
+            "variable to the folder where the web interface's "
+            "index.html is located.</p>"
+            "<p>Package Builders: to set a custom default at compile time, "
+            "#define PACKAGE_DATA_DIR in libtransmission/platform.c "
+            "or tweak tr_getClutchDir() by hand.</p>" );
+    }
+    else
+    {
+        char * pch;
+        char * subpath;
+        char * filename;
+
+        subpath = tr_strdup( req->uri + 18 );
+        if(( pch = strchr( subpath, '?' )))
+            *pch = '\0';
+
+        filename = *subpath
+            ? tr_strdup_printf( "%s%s%s", clutchDir, TR_PATH_DELIMITER_STR, subpath )
+            : tr_strdup_printf( "%s%s%s", clutchDir, TR_PATH_DELIMITER_STR, "index.html" );
+
+        serve_file( req, filename );
+
+        tr_free( filename );
+        tr_free( subpath );
+    }
 }