search

rnwood / smtp4dev

smtp4dev - the fake smtp email server for development and testing
BSD 3-Clause "New" or "Revised" License
3.07k stars 344 forks source link

Downloading attachment result in file called "content" instead of listed filename in email #113

Closed SchietSchijf closed 5 years ago

SchietSchijf commented 5 years ago

Hi,

I want to download an attachment (pdf in this case). If I click the download button, it will save the file as "content" without extention, while a proper filename with extention is provided.

Used build: Smtp4dev-win-x64-v3.1.0-ci0448

snippet of the raw message:

`Content-Type: application/octet-stream; name="Achzet, , Jeannette_364524.pdf" Content-Transfer-Encoding: base64 Content-Disposition: attachment

JVBERi0xLjUNCiW1tbW1DQoxIDAgb2JqDQo8PC9UeXBlL0NhdGFsb2cvUGFnZXMgMiAw<...>`

SchietSchijf commented 5 years ago

As far as I can see, the issue is not fixed with the new release (Smtp4dev-win-x64-v3.1.0-ci0470)

The attachment is still downloaded as "content". I tested this in Chrome and Firefox, same result.

I have attached the log of this email. message log.txt

SchietSchijf commented 5 years ago

Thanks for the quick fixes but I still get the filename "content", even with other attachments log of a txt attachment: message log2.txt

rnwood commented 5 years ago

Hi

Thanks for your example message. Sorry I didn't mean my initial commit to close this issue.

I'm not able to reproduce this with build 472 with your example message log2.txt: image

SchietSchijf commented 5 years ago

In firefox I get this: image Even displaying the email different. Chrome is displaying it the same as in my firefox screenshot I'm running smtp4dev on Windows 7

SchietSchijf commented 5 years ago

Anything I can do to give you more information? The actual content of the file is correct, if I append the file extention, it is shown properly.

rnwood commented 5 years ago

Can you confirm that last screenshot is when running build 472? The version is printed on the first line when running the .exe.

SchietSchijf commented 5 years ago

The issue has been resolved by the previous commit. I didn't see the fix because I made a shortcut on one of the older builds to have it listen on all interfaces but it still referred to the older build.

Now I get the same view as you have: image

Sorry for the trouble and thank you for fixing it!

rnwood commented 5 years ago

No problem. Thanks for reporting the issue.