MIME::QuotedPrint encode_qp emits LF instead of requested CRLF

Abierto Apto para principiantes
#1,341 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
88/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
java, perl
Área
backend

Línea de trabajo

Comienza en MIMEQuotedPrint.encodeQuotedPrintable, donde se emiten los saltos de línea normales de la entrada, y compara ese tratamiento con el uso existente de eol para los saltos de línea suaves. Añade cobertura específica para entradas CRLF y salidas CRLF solicitadas, y luego ejecuta la suite upstream t/basic.t tanto con ./jperl como con ./jperl --interpreter; se considera terminado cuando los bytes son exactamente CRLF y las 13 pruebas pasan.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

area:cpan-port area:unicode bug

Summary

PerlOnJava's bundled MIME::QuotedPrint provider emits LF line endings when
encode_qp is asked to use CRLF. This breaks Email::MIME::Encodings's
quoted-printable encoder, which is required to preserve RFC email line
endings.

Reproduction

Run the upstream Email::MIME::Encodings 1.317 test suite:

./jperl t/basic.t
./jperl --interpreter t/basic.t

The suite reports one failure in enc qp:

not ok 8 - enc qp
got:      'This is a test\nof various MIME=3Dstuff.=\n'
expected: 'This is a test\r\nof various MIME=3Dstuff.=\r\n'

The failure reproduces on both the JVM and interpreter backends. System Perl
passes the complete upstream suite: 13 tests successful.

A minimal provider-level reproducer is:

./jperl -MMIME::QuotedPrint -e 'my $x="a\r\nb"; my $y=encode_qp($x,"\r\n"); print unpack("H*",$y),"\n"'

The expected encoded bytes begin with 610d0a (a\\r\\n), but PerlOnJava
returns 610a (a\\n).

CPAN failure

The CPAN random-tester run 20260910-101908-8428 recorded FAIL for
Email::MIME::Encodings 1.317:

1/13 subtests failed; 1/2 test programs failed

The failing test is the distribution's stable t/basic.t test, assertion 8.
The distribution itself is pure Perl and has no external service or native
library prerequisite.

Technical cause

Email::MIME::Encodings::codec passes the requested EOL to
MIME::QuotedPrint::encode_qp:

return $sub->($what, $eol);

The PerlOnJava implementation in MIMEQuotedPrint.encodeQuotedPrintable
hardcodes LF when it encounters an input newline:

output.append(currentLine).append("\\n");

It should preserve the requested eol value (CRLF in this case). The same
implementation already uses eol for soft line breaks, so handling ordinary
newlines consistently with that parameter appears to be the missing piece.

Impact

Email and MIME libraries using PerlOnJava's quoted-printable provider can
produce non-conforming line endings even when explicitly requesting CRLF.
This can affect message serialization, canonicalization, signatures, and
interoperability with MIME consumers.

Suggested fix and regression coverage

Update MIMEQuotedPrint.encodeQuotedPrintable to append the requested EOL for
preserved newlines, while retaining existing binary-mode and CRLF-normalizing
behavior. Add a focused regression test that checks the exact bytes returned
by encode_qp for CRLF input and requested CRLF output, and run the
Email::MIME::Encodings test under both execution backends.

Lenguaje dominante
Perl
Estrellas
64
Forks
6
Merge medio
5 h 25 min
PR fusionados (30 d)
157

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de fglock/PerlOnJava

Todos los issues de fglock/PerlOnJava

Issues similares

Más issues de Perl

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.