Non-existant files and images throw 404, instead of 500, errors.
authorrich <rich>
Sun, 24 Oct 2004 11:34:37 +0000 (11:34 +0000)
committerrich <rich>
Sun, 24 Oct 2004 11:34:37 +0000 (11:34 +0000)
Bumped version for release.

debian/changelog
scripts/file.ml
scripts/image.ml

index eb013d0..63d9fe5 100644 (file)
@@ -1,4 +1,4 @@
-cocanwiki (1.3.2-4) unstable; urgency=low
+cocanwiki (1.3.2-5) unstable; urgency=low
 
   * Initial Release.
 
index eb45b71..28685df 100644 (file)
@@ -1,7 +1,7 @@
 (* COCANWIKI - a wiki written in Objective CAML.
  * Written by Richard W.M. Jones <rich@merjis.com>.
  * Copyright (C) 2004 Merjis Ltd.
- * $Id: file.ml,v 1.10 2004/10/24 08:48:38 rich Exp $
+ * $Id: file.ml,v 1.11 2004/10/24 11:34:37 rich Exp $
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -48,10 +48,13 @@ let run r (q : cgi) (dbh : Dbi.connection) hostid _ _ =
   sth#execute args;
 
   let data, mime_type, deleted =
-    match sth#fetch1 () with
-       [ `Binary data; `String mime_type; `Bool deleted ] ->
-         data, mime_type, deleted
-      | _ -> assert false in
+    try
+      (match sth#fetch1 () with
+          [ `Binary data; `String mime_type; `Bool deleted ] ->
+            data, mime_type, deleted
+        | _ -> assert false)
+    with
+       Not_found -> raise (HttpError cHTTP_NOT_FOUND) in
 
   (* If deleted, refuse to serve this file except if called from the
    * deleted files page.
index a625d78..0558950 100644 (file)
@@ -1,7 +1,7 @@
 (* COCANWIKI - a wiki written in Objective CAML.
  * Written by Richard W.M. Jones <rich@merjis.com>.
  * Copyright (C) 2004 Merjis Ltd.
- * $Id: image.ml,v 1.10 2004/10/24 08:48:38 rich Exp $
+ * $Id: image.ml,v 1.11 2004/10/24 11:34:37 rich Exp $
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -50,10 +50,13 @@ let run r (q : cgi) (dbh : Dbi.connection) hostid _ _ =
   sth#execute args;
 
   let data, mime_type, deleted =
-    match sth#fetch1 () with
-       [ `Binary data; `String mime_type; `Bool deleted ] ->
-         data, mime_type, deleted
-      | _ -> assert false in
+    try
+      (match sth#fetch1 () with
+          [ `Binary data; `String mime_type; `Bool deleted ] ->
+            data, mime_type, deleted
+        | _ -> assert false)
+    with
+       Not_found -> raise (HttpError cHTTP_NOT_FOUND) in
 
   (* If deleted, refuse to serve this image except if called from the
    * deleted images page.