HttpPostedFileBase.SaveAs working but no file uploaded and no exceptions

Question

First, here is my code:

private Shoe ProcessForm(Shoe shoe, HttpPostedFileBase image)
{
    try
    {
        shoe.Slug = CMSHelper.SanitizeTitle(shoe.Name);
        shoe.LastModification = DateTime.Now;

        if ((image != null) && (image.ContentLength > 0))
        {
            string fileName = String.Concat(shoe.ShoeId, Path.GetExtension(image.FileName));
            shoe.Image = fileName;

            string filePath = Path.Combine(Server.MapPath(shoe.ImagePath), fileName);
            image.SaveAs(filePath);
        }
    }
    catch (Exception e)
    {
        throw e;
    }

    return shoe;
}

Locally, this code works fine. Directories' permissions are fine. And it has worked before randomly on other servers (I tested this code on 4 or 5 different servers while I was testing VPS providers).

But if I try to run it from my home computer, everything passes alright, there's a file name saved in the database but no file is uploaded. And no exceptions are given!!!

I've been trying to fix this for almost three days and so much useless hours, please help me... I just don't see what's wrong with this...

_Please_ keep things like "C#/ASP.NET MVC 3: " out of your titles. Those are tags, and belong in the tags, not in your title. — John Saunders, Oct 05 '11 at 03:45
Also, don't use `try {} catch (Exception e){throw e;}`. If you're going to do that, then you're better off without the try/catch block at all. Your code will trash the stack trace. — John Saunders, Oct 05 '11 at 03:45
Alright for the first comment, I won't do it again. As for your second comment : What would you do? Ah I get what you mean, the exception will throw itself so it does not require me to catch it and throw it myself, right? Thanks for the comments. — Tommy B., Oct 05 '11 at 03:48
exceptions propagate themselves without our help. Also, your code will make it look like the exception came from the location of the `throw`. — John Saunders, Oct 05 '11 at 03:49
Maybe a stupid question, but are you sure the form tag ahve the enctype? Html.BeginForm("ProcessForm", "Home", null, FormMethod.Post, new { enctype = "multipart/form-data" }) — Iridio, Oct 05 '11 at 05:48
Yes: @using (Html.BeginForm("Edit", "Shoes", FormMethod.Post, new { enctype = "multipart/form-data" })) — Tommy B., Oct 17 '11 at 03:59

score 5 · Answer 1 · 2011-10-21T16:11:15.823

5

What is your view model? First of all replace (Shoe shoe, HttpPostedFileBase image) with (FormCollection formCollection). Put a breakpoint on that to see that you are getting all the submitted values. If you do then there is a problem with model binding and we'll need to address that.

Edit

Can you please put a breakpoint on

image.SaveAs(filePath);

Add a watch for fileName and filePath variables. I think that server path is not what you expect it to be, or maybe you are looking in the wrong folder all together. Tell us what are the values of those variables. Also make sure that no exception is thrown when you go past image.SaveAs(filePath);

edited Oct 21 '11 at 16:11

answered Oct 05 '11 at 08:57

I receive everything. The thing I find very weird is that when I test it LOCALLY everything works alright. I then, deploy it on the server and test it from the server (again locally) and it WORKS. But when I test it from home, on the server (remotely) there's a filename saved in the database but the file isn't saved on the server's harddrive... – Tommy B. Oct 17 '11 at 04:00
1

Have you checked event viewer on the server to see whether there are any access denied exceptions thrown when you try to write the file? For me it sounds like permission issue. If you don't have an event viewer, then try catching and logging that exception in a text file. – Oct 17 '11 at 08:30
I thought so at first and then I made the folder accessible to Everyone and never worked even though. I'll retry and check out the event viewer, Thanks. – Tommy B. Oct 17 '11 at 11:35
As I thought, NOTHING at all in the logs :-/ Any other ideas? – Tommy B. Oct 17 '11 at 12:33
Does a folder that you are saving to exists on a server? – Oct 17 '11 at 12:42
Yes it does. There is no error related to folders or permissions at all... :-S – Tommy B. Oct 17 '11 at 13:14

score 3 · Accepted Answer · answered Oct 19 '11 at 04:20

I finally did a workaround which is doing very fine. I even asked at my job and everyone said there was nothing wrong at all. So screw it here's what I did:

Instead of calling .SaveAs() I made a method which is :

public static void WriteFileFromStream(Stream stream, string toFile)
{
    using (FileStream fileToSave = new FileStream(toFile, FileMode.Create))
    {
        stream.CopyTo(fileToSave);
    }
}

I call it like this:

CMSHelper.WriteFileFromStream(image.InputStream, filePath);

And that's it.

score 2 · Answer 3 · answered Dec 31 '13 at 06:47

image.FileName contains the path data and is selected by

Path.Combine(Server.MapPath(shoe.ImagePath), fileName)

Try something like:

string pathForSaving = Server.MapPath("~/Upload/Images");
string uploadFileName = System.IO.Path.GetFileName(image.FileName);
string uploadFilePathAndName = Path.Combine(pathForSaving, uploadFileName);
image.SaveAs(uploadFilePathAndName);

score 0 · Answer 4 · answered Jul 11 '13 at 21:22

0

Make sure you are using enctype="multipart/form-data" in the form

http://www.prideparrot.com/blog/archive/2012/8/uploading_and_returning_files

answered Jul 11 '13 at 21:22

Chau

1

score 0 · Answer 5 · answered Mar 29 '16 at 17:07

I also ran into the same issue posted on this thread and after a few tries of uploading a file, I refreshed the folder where the files were suppose to be and finally I saw them there. Not sure why it didn't show up the first time after the first refresh but it probably has something to do with refreshing the folder. I did the refresh from within Visual Studio and I also had the "Show All Files" button on.

score 0 · Answer 6 · answered Oct 17 '11 at 13:25

0

How are you setting shoe.ImagePath? Your issue feels like Server.MapPath is returning a value that you are not expecting.

answered Oct 17 '11 at 13:25

Maciej

2,147
1
17
27

it is a simple property set in my model like this: ImagePath = "/Assets/Uploads/Shoes"; And I tested the Server.MapPath and it returns a perfect existing path ... :-S – Tommy B. Oct 17 '11 at 13:40

score 0 · Answer 7 · answered Oct 24 '11 at 03:45

Did you check to see if you have multiple web servers? Like a load balancer perhaps. It might not be throwing an exception because there isn't actually an exception and the file does actually exist somewhere, just not on the server you believe it is on. Just a thought (it has happened to me before).

HttpPostedFileBase.SaveAs working but no file uploaded and no exceptions

7 Answers7