neojn / mailsend

Automatically exported from code.google.com/p/mailsend
0 stars 0 forks source link

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

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
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 reported on code.google.com by skalogry...@gmail.com on 27 Oct 2013 at 4:31

GoogleCodeExporter commented 8 years ago
this is not a defect, of course.

Original comment by skalogry...@gmail.com on 27 Oct 2013 at 4:32

GoogleCodeExporter commented 8 years ago

Original comment by muq...@gmail.com on 27 Oct 2013 at 4:34

GoogleCodeExporter commented 8 years ago
The support is available with the latest source in svn.

Original comment by muq...@gmail.com on 27 Oct 2013 at 7:42

GoogleCodeExporter commented 8 years ago
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.

Original comment by skalogry...@gmail.com on 4 Nov 2013 at 2:08

GoogleCodeExporter commented 8 years ago
so far, works nicely. 
I'd suggest to close the issue!
thank you very much.

Original comment by skalogry...@gmail.com on 4 Nov 2013 at 2:18

GoogleCodeExporter commented 8 years ago
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).

Original comment by skalogry...@gmail.com on 4 Nov 2013 at 6:08

GoogleCodeExporter commented 8 years ago
Yes, memset(a,0,sizeof(Attachment));  should have been there. Good find!

Original comment by muq...@gmail.com on 7 Nov 2013 at 12:56

GoogleCodeExporter commented 8 years ago

Original comment by muq...@gmail.com on 7 Dec 2013 at 5:39