[ { "Id": "237044", "ThreadId": "69638", "Html": "

I can create a zip file if I hard code the file path.

\r\n

Zip.AddFile(C:\\filename.pdf", "")
Zip.AddFile("C:\\Test.txt", "")
Zip.Save(Response.OutputStream)

\r\n

This works! 

\r\n

If I place the same values into a file array, it fails

\r\n

filenames = fileList.ToArray()
Zip.AddFile(filenames(0), "")
Zip.AddFile(filenames(1), "")
Zip.Save(Response.OutputStream)

\r\n

I have my file name and path stored in a database.  I need to read the file name and path and create a zip file to be downloaded.

\r\n

Yes, the values are the same in the filenames array. 

\r\n

What am I doing wrong?

\r\n

I have trying reading the file into a memorystream.  It works if the path is hard codes, but once I use variables the zip file is damaged.  It says the file name are too long.  I can open it, but each file name has .... at the end.  Again, if I put the hard code the file path everything is ok. 

\r\n

The files are stored on a network share.  I have full permission.  Everything works ok if I hard code the path (type it in).  If I use variables, the zip file is damaged and can not view it.  It has to be something with how my variables are defined.  I have tried Lists, arrays, reading from a database and any other method I could find or think of. 

", "PostedDate": "2009-09-21T13:59:17-07:00", "UserRole": null, "MarkedAsAnswerDate": null }, { "Id": "237155", "ThreadId": "69638", "Html": "

I suggest you employ:

\r\n\r\n

You may be including these things but you don't show them.

\r\n

I don't know what this means:  "It says the file name are too long.  I can open it, but each file name has .... at the end."   How do you view the value of the filename to see the ... ?  What tool are you using to view the file name, and what exactly is the "file name" in this case - a string variable in your program?  A property on the ZipFile object?  something else?

\r\n

I can't imagine the problem but I think maybe the variables do not hold the values you think they do.  There's an assumption you are making which seems safe and correct but isn't.   Do you know how to run your app in the debugger?  Maybe check the value of the filenames(0) and filenames(1) things, in the debugger, during a run.

\r\n

Good luck!

", "PostedDate": "2009-09-21T23:49:45.67-07:00", "UserRole": null, "MarkedAsAnswerDate": null }, { "Id": "237289", "ThreadId": "69638", "Html": "

The problem seems to happen with with UNC paths (not local).   If I use a local path C:\\ instead of \\\\server\\share it works with variables. 

\r\n

With \\\\server\\share in a foreach loop with variables.  I am getting the path from a database.  If I hard code the \\\\server\\share it works correctly.  I have stepped through the code with debugger and verfied the paths by copy and pasting the values into windows explorer.  I know it is correct.

\r\n

here is one of the for each statements it is failing on:

\r\n

Using ZIp As New ZipFile

\r\n

       HttpContext.Current.Response.Clear()
       HttpContext.Current.Response.BufferOutput = False
       HttpContext.Current.Response.ContentType = "application/zip"
       HttpContext.Current.Response.AddHeader("content-disposition", "attachment; filename=" + archiveName)
       Using dataReader As IDataReader = db.ExecuteReader(command)

\r\n

       While (dataReader.Read())        
         Zip.AddFile(dataReader("FileDirectory") + "\\" + dataReader("FileName"), "")
       End While
       ZIp.Save(HttpContext.Current.Response.OutputStream)
      End Using
End Using

\r\n

I am using winrar to view the archieve.  It also fails with Windows XP built in zip handler. 

\r\n

Winrar results:
The zip is the correct size.  The files inside the zip show the correct size, but the Type shows "File ???"  The filename should be 0_3boxes.jpg, but it shows it as 0_3boxes.jpg ...

\r\n

Windows XP Zip Handler:
The compressed (zipped) Folder is invalid or corrupt.

\r\n

The problem does not exist if I use a local path in the database.  I think maybe the files are not being copied or the operation is timing out before the files are completely copied. 

", "PostedDate": "2009-09-22T06:10:39.987-07:00", "UserRole": null, "MarkedAsAnswerDate": null }, { "Id": "237297", "ThreadId": "69638", "Html": "

If the files did not exist, the zip class would through an exception.

", "PostedDate": "2009-09-22T06:20:55.54-07:00", "UserRole": null, "MarkedAsAnswerDate": null }, { "Id": "237300", "ThreadId": "69638", "Html": "

That why you need breaks when you code.  The database variable had an extra 100 spaces in it.  A simple trim() statement fixed that. 

", "PostedDate": "2009-09-22T06:32:52.553-07:00", "UserRole": null, "MarkedAsAnswerDate": null }, { "Id": "237338", "ThreadId": "69638", "Html": "

Great, glad you found it.

\r\n

I'm trying to decide whether DotNetZip should do the Trim() for you or not.  At this point I'm not sure.  A filename on Windows cannot end with a space.  I'm going to think about this a little.

\r\n

 

", "PostedDate": "2009-09-22T07:53:39.397-07:00", "UserRole": null, "MarkedAsAnswerDate": null }, { "Id": "237339", "ThreadId": "69638", "Html": "This discussion has been copied to a work item. Click here to go to the work item and continue the discussion.", "PostedDate": "2009-09-22T07:55:07.117-07:00", "UserRole": null, "MarkedAsAnswerDate": null }, { "Id": "237380", "ThreadId": "69638", "Html": "

I was joining strings together to form a path.   filepath + "\\" + filename.   I needed to trim the filepath and filename variables.  It should be left to the user of the library to trim the strings.

\r\n

Thanks for your great work on the library and your help.

", "PostedDate": "2009-09-22T08:53:18.32-07:00", "UserRole": null, "MarkedAsAnswerDate": null } ]