2009-06-04 Rolf Bjarne Kvinge <RKvinge@novell.com>

	* GetFile.aspx.cs: Don't use WriteFile, deleting the file
	right after breaks apache.

svn path=/trunk/monkeywrench/; revision=135373

MonkeyWrench.Web.UI/ChangeLog

@@ -1,3 +1,8 @@
+2009-06-04  Rolf Bjarne Kvinge  <RKvinge@novell.com>
+
+	* GetFile.aspx.cs: Don't use WriteFile, deleting the file
+	right after breaks apache.
+
 2009-06-03  Rolf Bjarne Kvinge  <RKvinge@novell.com>
 
 	* ViewWorkTable.aspx.designer.cs: Updated.

MonkeyWrench.Web.UI/GetFile.aspx.cs

@@ -37,7 +37,19 @@ protected void Page_Load (object sender, EventArgs e)
 				Response.AppendHeader ("Content-Disposition", web.ResponseHeaders ["Content-Disposition"]);
 				Response.AppendHeader ("Content-Encoding", web.ResponseHeaders ["Content-Encoding"]);
 
-				Response.WriteFile (tmpfile);
+				byte [] buffer = new byte [1024];
+				int read;
+				using (FileStream fs = new FileStream (tmpfile, FileMode.Open, FileAccess.Read, FileShare.Read)) {
+					using (StreamReader reader = new StreamReader (fs)) {
+						while ((read = fs.Read (buffer, 0, buffer.Length)) != 0) {
+							Response.OutputStream.Write (buffer, 0, read);
+						}
+					}
+				}
+				// This would be the best I guess, but I get intermittent failures with 
+				// " [error] command failed: failed to send file (file data)" 
+				// in apache's error log
+				// Response.WriteFile (tmpfile);
 				Response.Flush ();
 				Response.Close ();
 			}