Gossamer Forum
Home : Products : Gossamer Mail : Discussion :

Error forwarding attachments...

Quote Reply
Error forwarding attachments...
Good day everyone,

A problem has come to my attention. When users forwarding a message that contains an attachment whose filename contains spaces generates the following error:

We are unable to locate the attachment!

Clicking on the attachment link in the list yields:

The filename entered is invalid. Please limit your attachment filenames to letters and numbers only (x:/xxxxx/cgi-bin/data/users/xxxx/x/xxx/upload/5d6b5877c7eb99758cd632be6db8fe04/Eppco%20site%20100.jpg).

Everything works fine from Read. Only forwarding has the problem.

Given that attachment filenames are out of the users control and saving the attachment locally and re-attaching it would be an inconvienance, how difficult will it be to implement functionality to convert spaces to underscores in the processing of attachments, and/or the option to forward the origional encoded message as an attachment?

GMail version 2.0.6



Regards,

Jason
Quote Reply
Re: [dig.it] Error forwarding attachments... In reply to
Hi,

We also noticed that same problem happen and would like to see a solution for it.

Thanks,

Frank
Quote Reply
Re: [dig.it] Error forwarding attachments... In reply to
If you take a look at Compose.pm, line ~149, you'll see:
Code:
$write = "$attchdir/" . ($_->{msgdata_suggestedname} || $i++);
The filename there should be escaped, so it should be:
Code:
$write = "$attchdir/" . $IN->escape($_->{msgdata_suggestedname} || $i++);

Adrian
Quote Reply
Re: [brewt] Error forwarding attachments... In reply to
Works great! Thanks.

-Jase
Quote Reply
Re: [brewt] Error forwarding attachments... In reply to
In Reply To:
If you take a look at Compose.pm, line ~149, you'll see:
Code:
$write = "$attchdir/" . ($_->{msgdata_suggestedname} || $i++);
The filename there should be escaped, so it should be:
Code:
$write = "$attchdir/" . $IN->escape($_->{msgdata_suggestedname} || $i++);



Hi Adrian,

Would this fix be the same for Version: 2.0.2-Upgrade?

Yes - I know it's time I upgraded! Wink - working on that now.

r.
Quote Reply
Re: [ryel01] Error forwarding attachments... In reply to
Yup, it looks like it's on the same line too.

Adrian