muquit / mailsend

A program to send mail via SMTP from command line
Other
295 stars 68 forks source link

feature request: -attach, specify a different name (that file path) for the attachment #27

Closed muquit closed 10 years ago

muquit commented 10 years ago

From skalogry...@gmail.com on October 27, 2013 00:31:11

right now, the attached file name, will default to a physical name of a file (as seen on the storage). please add the ability (the 4th option of -attachment?) to specify the attachment name: example: -attach /files/doc/CRAZYGUID123123128937123.txt,text/plain,a,uploadlog.txt

(i believe it makes no sense for "inline" attachments)

Original issue: http://code.google.com/p/mailsend/issues/detail?id=27

muquit commented 10 years ago

From skalogry...@gmail.com on October 26, 2013 21:32:07

this is not a defect, of course.

muquit commented 10 years ago

From muq...@gmail.com on October 27, 2013 09:34:05

Status: Started

muquit commented 10 years ago

From muq...@gmail.com on October 27, 2013 12:42:28

The support is available with the latest source in svn.

muquit commented 10 years ago

From skalogry...@gmail.com on November 04, 2013 06:08:03

ok.. so building from source.

It's worth adding to the "Windows installation" page, that, if starting building from scratch, openSSL lib (static libraries) are required. If not no static libraries are available can be built from OpenSSL source, but building SSL source requires Perl installed (for preparing make environment)

built pretty much successful, testing now.

muquit commented 10 years ago

From skalogry...@gmail.com on November 04, 2013 06:18:00

so far, works nicely. I'd suggest to close the issue! thank you very much.

muquit commented 10 years ago

From skalogry...@gmail.com on November 04, 2013 10:08:49

i think *attachment_name needs to be initialized to null, when allocating the header. (otherwise the code fails later on, when it checks for the attachment_name).

muquit commented 10 years ago

From muq...@gmail.com on November 06, 2013 16:56:29

Yes, memset(a,0,sizeof(Attachment)); should have been there. Good find!

muquit commented 10 years ago

From muq...@gmail.com on December 06, 2013 21:39:47

Status: Done