[PEAR-BUG] Bug #15888 [Fbk->NoF]: attachment name/filename header lacks semicolon separator

Edit report at http://pear.php.net/bugs/bug.php?id=15888&edit=1

ID: 15888
Updated by: [email protected]
Reported By: alec at alec dot pl
Summary: attachment name/filename header lacks semicolon
separator
Status: No Feedback
Type: Bug
Package: Mail_Mime
PHP Version: 5.2.8
Roadmap Versions:
New Comment:

This is still not fixed. One line patch for _buildHeaderParam():
- $headers = implode(MAIL_MIMEPART_CRLF, $headers) . ';';
+ $headers = implode(';' . MAIL_MIMEPART_CRLF, $headers) . ';';


Previous Comments:
------------------------------------------------------------------------

[2009-08-25 07:44:20] cweiske

-Status: Feedback
+Status: No Feedback
No feedback was provided. The bug is being suspended because
we assume that you are no longer experiencing the problem.
If this is not the case and you are able to provide the
information that was requested earlier, please do so and
change the status of the bug back to "Open". Thank you.

------------------------------------------------------------------------

[2009-02-11 06:32:54] doconnor

-Status: Open
+Status: Feedback

We are unable to reproduce the bug as reported.
Please provide a simple script (10 lines or less)
or a link to an external script that can be used
to reproduce the bug.

If you can provide the script, feel free to add it
to this bug and change the status back to "Open".

Thank you for your interest in PEAR.


Can you attach a small reproducable executable test case?

------------------------------------------------------------------------

[2009-02-10 07:44:01] alec

Description:
------------
Now:
Content-Disposition: attachment;
filename*0*="UTF-8''very long long long filename filename filename foo
bar"
filename*1*="boo baz bang";

Should be:
Content-Disposition: attachment;
filename*0*="UTF-8''very long long long filename filename filename foo
bar";
filename*1*="boo baz bang";


Test script:
---------------


Expected result:
----------------


Actual result:
--------------

------------------------------------------------------------------------

Edit report at http://pear.php.net/bugs/bug.php?id=15888&edit=1

ID: 15888
Updated by: [email protected]
Reported By: alec at alec dot pl
Summary: attachment name/filename header lacks semicolon
separator
-Status: No Feedback
+Status: Feedback
Type: Bug
Package: Mail_Mime
PHP Version: 5.2.8
Roadmap Versions:
New Comment:

-Status: No Feedback
+Status: Feedback
Alec, could you please provide a test script that demonstrates the
problem? Without it, we cannot do anything.


Previous Comments:
------------------------------------------------------------------------

[2009-08-25 09:27:09] alec

This is still not fixed. One line patch for _buildHeaderParam():
- $headers = implode(MAIL_MIMEPART_CRLF, $headers) . ';';
+ $headers = implode(';' . MAIL_MIMEPART_CRLF, $headers) . ';';

------------------------------------------------------------------------

[2009-08-25 07:44:20] cweiske

-Status: Feedback
+Status: No Feedback
No feedback was provided. The bug is being suspended because
we assume that you are no longer experiencing the problem.
If this is not the case and you are able to provide the
information that was requested earlier, please do so and
change the status of the bug back to "Open". Thank you.

------------------------------------------------------------------------

[2009-02-11 06:32:54] doconnor

-Status: Open
+Status: Feedback

We are unable to reproduce the bug as reported.
Please provide a simple script (10 lines or less)
or a link to an external script that can be used
to reproduce the bug.

If you can provide the script, feel free to add it
to this bug and change the status back to "Open".

Thank you for your interest in PEAR.


Can you attach a small reproducable executable test case?

------------------------------------------------------------------------

[2009-02-10 07:44:01] alec

Description:
------------
Now:
Content-Disposition: attachment;
filename*0*="UTF-8''very long long long filename filename filename foo
bar"
filename*1*="boo baz bang";

Should be:
Content-Disposition: attachment;
filename*0*="UTF-8''very long long long filename filename filename foo
bar";
filename*1*="boo baz bang";


Test script:
---------------


Expected result:
----------------


Actual result:
--------------

------------------------------------------------------------------------

Edit report at http://pear.php.net/bugs/bug.php?id=15888&edit=1

ID: 15888
Comment by: [email protected]
Reported By: alec at alec dot pl
Summary: attachment name/filename header lacks semicolon
separator
Status: Feedback
Type: Bug
Package: Mail_Mime
PHP Version: 5.2.8
Roadmap Versions:
New Comment:

<?php

include('Mail/mime.php');

$filename = 'long name with spaces, very long long long long.txt';

$mimepart = new Mail_mimePart('');

echo $mimepart->_buildHeaderParam('filename', $filename, 'UTF-8');

?>


Previous Comments:
------------------------------------------------------------------------

[2009-08-25 09:30:02] cweiske

-Status: No Feedback
+Status: Feedback
Alec, could you please provide a test script that demonstrates the
problem? Without it, we cannot do anything.

------------------------------------------------------------------------

[2009-08-25 09:27:09] alec

This is still not fixed. One line patch for _buildHeaderParam():
- $headers = implode(MAIL_MIMEPART_CRLF, $headers) . ';';
+ $headers = implode(';' . MAIL_MIMEPART_CRLF, $headers) . ';';

------------------------------------------------------------------------

[2009-08-25 07:44:20] cweiske

-Status: Feedback
+Status: No Feedback
No feedback was provided. The bug is being suspended because
we assume that you are no longer experiencing the problem.
If this is not the case and you are able to provide the
information that was requested earlier, please do so and
change the status of the bug back to "Open". Thank you.

------------------------------------------------------------------------

[2009-02-11 06:32:54] doconnor

-Status: Open
+Status: Feedback

We are unable to reproduce the bug as reported.
Please provide a simple script (10 lines or less)
or a link to an external script that can be used
to reproduce the bug.

If you can provide the script, feel free to add it
to this bug and change the status back to "Open".

Thank you for your interest in PEAR.


Can you attach a small reproducable executable test case?

------------------------------------------------------------------------

[2009-02-10 07:44:01] alec

Description:
------------
Now:
Content-Disposition: attachment;
filename*0*="UTF-8''very long long long filename filename filename foo
bar"
filename*1*="boo baz bang";

Should be:
Content-Disposition: attachment;
filename*0*="UTF-8''very long long long filename filename filename foo
bar";
filename*1*="boo baz bang";


Test script:
---------------


Expected result:
----------------


Actual result:
--------------

------------------------------------------------------------------------

Edit report at http://pear.php.net/bugs/bug.php?id=15888&edit=1

ID: 15888
Comment by: [email protected]
Reported By: alec at alec dot pl
Summary: attachment name/filename header lacks semicolon
separator
Status: Feedback
Type: Bug
Package: Mail_Mime
PHP Version: 5.2.8
Roadmap Versions:
New Comment:

Also my bulk patch in #12411 addresses this issue.


Previous Comments:
------------------------------------------------------------------------

[2009-08-25 09:41:00] alec

<?php

include('Mail/mime.php');

$filename = 'long name with spaces, very long long long long.txt';

$mimepart = new Mail_mimePart('');

echo $mimepart->_buildHeaderParam('filename', $filename, 'UTF-8');

?>

------------------------------------------------------------------------

[2009-08-25 09:30:02] cweiske

-Status: No Feedback
+Status: Feedback
Alec, could you please provide a test script that demonstrates the
problem? Without it, we cannot do anything.

------------------------------------------------------------------------

[2009-08-25 09:27:09] alec

This is still not fixed. One line patch for _buildHeaderParam():
- $headers = implode(MAIL_MIMEPART_CRLF, $headers) . ';';
+ $headers = implode(';' . MAIL_MIMEPART_CRLF, $headers) . ';';

------------------------------------------------------------------------

[2009-08-25 07:44:20] cweiske

-Status: Feedback
+Status: No Feedback
No feedback was provided. The bug is being suspended because
we assume that you are no longer experiencing the problem.
If this is not the case and you are able to provide the
information that was requested earlier, please do so and
change the status of the bug back to "Open". Thank you.

------------------------------------------------------------------------

[2009-02-11 06:32:54] doconnor

-Status: Open
+Status: Feedback

We are unable to reproduce the bug as reported.
Please provide a simple script (10 lines or less)
or a link to an external script that can be used
to reproduce the bug.

If you can provide the script, feel free to add it
to this bug and change the status back to "Open".

Thank you for your interest in PEAR.


Can you attach a small reproducable executable test case?

------------------------------------------------------------------------

The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
http://pear.php.net/bugs/bug.php?id=15888

Edit report at http://pear.php.net/bugs/bug.php?id=15888&edit=1

ID: 15888
Updated by: [email protected]
Reported By: alec at alec dot pl
Summary: attachment name/filename header lacks semicolon
separator
Status: Feedback
Type: Bug
Package: Mail_Mime
PHP Version: 5.2.8
Roadmap Versions:
New Comment:

Sylpheed does not use filename*x* at all:

--Multipart=_Tue__25_Aug_2009_09_54_22_+0200_+xAiUIxEQre8wtG5
Content-Type: text/plain;
name="das hier ist ein sehr langer dateiname - und zwar so lang, dass
er nicht in eine zeile beim mime-zeug passt. was machen wir also hier
fragezeichen - einfach nichts, oder doch etwas fragezeichen.txt"
Content-Disposition: attachment;
filename="das hier ist ein sehr langer dateiname - und zwar so lang,
dass
er nicht in eine zeile beim mime-zeug passt. was machen wir also hier
fragezeichen - einfach nichts, oder doch etwas fragezeichen.txt"
Content-Transfer-Encoding: 7bit


Previous Comments:
------------------------------------------------------------------------

[2009-08-25 09:43:09] alec

Also my bulk patch in #12411 addresses this issue.

------------------------------------------------------------------------

[2009-08-25 09:41:00] alec

<?php

include('Mail/mime.php');

$filename = 'long name with spaces, very long long long long.txt';

$mimepart = new Mail_mimePart('');

echo $mimepart->_buildHeaderParam('filename', $filename, 'UTF-8');

?>

------------------------------------------------------------------------

[2009-08-25 09:30:02] cweiske

-Status: No Feedback
+Status: Feedback
Alec, could you please provide a test script that demonstrates the
problem? Without it, we cannot do anything.

------------------------------------------------------------------------

[2009-08-25 09:27:09] alec

This is still not fixed. One line patch for _buildHeaderParam():
- $headers = implode(MAIL_MIMEPART_CRLF, $headers) . ';';
+ $headers = implode(';' . MAIL_MIMEPART_CRLF, $headers) . ';';

------------------------------------------------------------------------

[2009-08-25 07:44:20] cweiske

-Status: Feedback
+Status: No Feedback
No feedback was provided. The bug is being suspended because
we assume that you are no longer experiencing the problem.
If this is not the case and you are able to provide the
information that was requested earlier, please do so and
change the status of the bug back to "Open". Thank you.

------------------------------------------------------------------------

The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
http://pear.php.net/bugs/bug.php?id=15888

Edit report at http://pear.php.net/bugs/bug.php?id=15888&edit=1

ID: 15888
Updated by: [email protected]
Reported By: alec at alec dot pl
Summary: attachment name/filename header lacks semicolon
separator
Status: Feedback
Type: Bug
Package: Mail_Mime
PHP Version: 5.2.8
Roadmap Versions:
New Comment:

http://trac.roundcube.net/ticket/1485727
http://groups.google.com/group/opera.mail+news/browse_thread/thread/af353c9052ce59ed

You are right. The standard says that distinct parts of an header need
to be separated by a semicolon. Since filename*0* and filename*1* are
different parts, even if they belong together, they need to have
semicolons between them.


Previous Comments:
------------------------------------------------------------------------

[2009-08-25 09:55:32] cweiske

Sylpheed does not use filename*x* at all:

--Multipart=_Tue__25_Aug_2009_09_54_22_+0200_+xAiUIxEQre8wtG5
Content-Type: text/plain;
name="das hier ist ein sehr langer dateiname - und zwar so lang, dass
er nicht in eine zeile beim mime-zeug passt. was machen wir also hier
fragezeichen - einfach nichts, oder doch etwas fragezeichen.txt"
Content-Disposition: attachment;
filename="das hier ist ein sehr langer dateiname - und zwar so lang,
dass
er nicht in eine zeile beim mime-zeug passt. was machen wir also hier
fragezeichen - einfach nichts, oder doch etwas fragezeichen.txt"
Content-Transfer-Encoding: 7bit

------------------------------------------------------------------------

[2009-08-25 09:43:09] alec

Also my bulk patch in #12411 addresses this issue.

------------------------------------------------------------------------

[2009-08-25 09:41:00] alec

<?php

include('Mail/mime.php');

$filename = 'long name with spaces, very long long long long.txt';

$mimepart = new Mail_mimePart('');

echo $mimepart->_buildHeaderParam('filename', $filename, 'UTF-8');

?>

------------------------------------------------------------------------

[2009-08-25 09:30:02] cweiske

-Status: No Feedback
+Status: Feedback
Alec, could you please provide a test script that demonstrates the
problem? Without it, we cannot do anything.

------------------------------------------------------------------------

[2009-08-25 09:27:09] alec

This is still not fixed. One line patch for _buildHeaderParam():
- $headers = implode(MAIL_MIMEPART_CRLF, $headers) . ';';
+ $headers = implode(';' . MAIL_MIMEPART_CRLF, $headers) . ';';

------------------------------------------------------------------------

The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
http://pear.php.net/bugs/bug.php?id=15888

Edit report at http://pear.php.net/bugs/bug.php?id=15888&edit=1

ID: 15888
Updated by: [email protected]
Reported By: alec at alec dot pl
Summary: attachment name/filename header lacks semicolon
separator
-Status: Feedback
+Status: Verified
Type: Bug
Package: Mail_Mime
PHP Version: 5.2.8
Roadmap Versions:
New Comment:

-Status: Feedback
+Status: Verified



Previous Comments:
------------------------------------------------------------------------

[2009-08-25 10:00:23] cweiske

http://trac.roundcube.net/ticket/1485727
http://groups.google.com/group/opera.mail+news/browse_thread/thread/af353c9052ce59ed

You are right. The standard says that distinct parts of an header need
to be separated by a semicolon. Since filename*0* and filename*1* are
different parts, even if they belong together, they need to have
semicolons between them.

------------------------------------------------------------------------

[2009-08-25 09:55:32] cweiske

Sylpheed does not use filename*x* at all:

--Multipart=_Tue__25_Aug_2009_09_54_22_+0200_+xAiUIxEQre8wtG5
Content-Type: text/plain;
name="das hier ist ein sehr langer dateiname - und zwar so lang, dass
er nicht in eine zeile beim mime-zeug passt. was machen wir also hier
fragezeichen - einfach nichts, oder doch etwas fragezeichen.txt"
Content-Disposition: attachment;
filename="das hier ist ein sehr langer dateiname - und zwar so lang,
dass
er nicht in eine zeile beim mime-zeug passt. was machen wir also hier
fragezeichen - einfach nichts, oder doch etwas fragezeichen.txt"
Content-Transfer-Encoding: 7bit

------------------------------------------------------------------------

[2009-08-25 09:43:09] alec

Also my bulk patch in #12411 addresses this issue.

------------------------------------------------------------------------

[2009-08-25 09:41:00] alec

<?php

include('Mail/mime.php');

$filename = 'long name with spaces, very long long long long.txt';

$mimepart = new Mail_mimePart('');

echo $mimepart->_buildHeaderParam('filename', $filename, 'UTF-8');

?>

------------------------------------------------------------------------

[2009-08-25 09:30:02] cweiske

-Status: No Feedback
+Status: Feedback
Alec, could you please provide a test script that demonstrates the
problem? Without it, we cannot do anything.

------------------------------------------------------------------------

The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
http://pear.php.net/bugs/bug.php?id=15888