Issue with cyrillic encoded attachments from mailbox

[akstis-typer]

Code of Conduct

• I agree to follow this project's Code of Conduct

Disable plugins

• I reproduce the issue with all plugins disabled

Is there an existing issue for this?

• I have searched the existing issues

Version

11.0.4

Bug description

When mailcollector collecting mail from mailbox - csv files that were encoded in Windows1251 encoding will be processed by Mailcollector so they'll no longer have valid encoding. I find issue in src/Mailcollector.php in getDecodedContent method. The files that doesn't include charset info get decoded with ISO-8859-1
The issue is that - the files that mailcollector process will have corrupted encoding - so when you download them from GLPI - they no longer valid
I provide a PR that will solve this issue

Relevant log output

Page URL

No response

Steps To reproduce

1. Configure mailcollector in GLPI
2. Send an email with attachment encoded with Windows1251 for example
3. Recieve a ticket, with attachment that'll have broken encoding
   Note: not every encoded file will be corrupted, only files that doesn't have charset data in it

Your GLPI setup information

No response

Anything else?

No response

[Aleko-SaR]

The following change in the file worked for me Mailcollector.php:
/**

• Retrieve properly decoded content

• @param Part $part Message Part

• @return string
  */
  public function getDecodedContent(Part $part)
  {
  $contents = $part->getContent();

  $encoding = null;
  if (isset($part->contentTransferEncoding)) {
  $encoding = $part->contentTransferEncoding;
  }

  // Декодирование содержимого в соответствии с Content-Transfer-Encoding
  switch (strtolower(trim($encoding))) {
  case 'base64':
  $contents = base64_decode($contents, true);
  if ($contents === false) {
  // Если декодирование не удалось, пробуем с игнорированием некорректных символов
  $contents = base64_decode($contents, false);
  }
  break;
  case 'quoted-printable':
  $contents = quoted_printable_decode($contents);
  break;
  case '7bit':
  case '8bit':
  case 'binary':
  default:
  // возвращаем как есть
  break;
  }

  // Проверяем, является ли часть текстовой
  if (
  !$part->getHeaders()->has('content-type')
  || !(($content_type = $part->getHeader('content-type')) instanceof ContentType)
  || preg_match('/^text//', $content_type->getType()) !== 1
  ) {
  // Для нетекстовых вложений (изображения, архивы и т.д.) возвращаем как есть
  return $contents;
  }

  $charset = $content_type->getParameter('charset');

  // Определяем кодировку, если не указана
  if ($charset === null) {
  // Пробуем определить кодировку автоматически
  $detectedEncoding = mb_detect_encoding($contents, ['UTF-8', 'Windows-1251', 'KOI8-R', 'ISO-8859-5', 'CP866'], true);
  $charset = $detectedEncoding ?: 'Windows-1251'; // По умолчанию Windows-1251 для кириллицы
  }

  // Нормализуем название кодировки
  $charset = $this->normalizeCharset($charset);
  $charsetUpper = strtoupper($charset);

  // Если кодировка уже UTF-8, возвращаем как есть
  if ($charsetUpper === 'UTF-8' || $charsetUpper === 'UTF8') {
  return $contents;
  }

  // Специальная обработка для кириллических кодировок
  if (in_array($charsetUpper, ['WINDOWS-1251', 'CP1251', 'WINDOWS1251'])) {
  return $this->convertToUtf8($contents, 'Windows-1251');
  }

  // Специальная обработка для корейских кодировок
  if (in_array(strtolower($charset), ['ks_c_5601-1987', 'ks_c_5601-1989'])) {
  $charset = 'UHC';
  }

  // Специальная обработка для ISO-8859-8-i
  if ($charsetUpper === 'ISO-8859-8-I') {
  $charset = 'ISO-8859-8';
  }

  // Конвертация в UTF-8
  return $this->convertToUtf8($contents, $charset);
  }

/**

• Нормализует название кодировки

• @param string $charset

• @return string
  */
  private function normalizeCharset($charset)
  {
  if (empty($charset)) {
  return 'UTF-8';
  }

  $charset = trim($charset);
  $charsetUpper = strtoupper($charset);

  // Маппинг различных вариантов написания кодировок
  $charsetMap = [
  'WINDOWS1251' => 'Windows-1251',
  'WINDOWS-1251' => 'Windows-1251',
  'CP1251' => 'Windows-1251',
  'CP-1251' => 'Windows-1251',
  'WINDOWS1252' => 'Windows-1252',
  'WINDOWS-1252' => 'Windows-1252',
  'CP1252' => 'Windows-1252',
  'CP-1252' => 'Windows-1252',
  'KOI8R' => 'KOI8-R',
  'KOI8-R' => 'KOI8-R',
  'KOI8U' => 'KOI8-U',
  'KOI8-U' => 'KOI8-U',
  'UTF8' => 'UTF-8',
  'UTF-8' => 'UTF-8',
  ];

  if (isset($charsetMap[$charsetUpper])) {
  return $charsetMap[$charsetUpper];
  }

  return $charset;
  }

/**

• Конвертирует содержимое в UTF-8

• @param string $contents

• @param string $charset

• @return string
  */
  private function convertToUtf8($contents, $charset)
  {
  // Пробуем конвертировать через mbstring
  if (in_array(strtoupper($charset), array_map('strtoupper', mb_list_encodings()))) {
  $converted = mb_convert_encoding($contents, 'UTF-8', $charset);
  if ($converted !== false) {
  return $converted;
  }
  }

  // Пробуем конвертировать через iconv
  try {
  // Сначала пробуем с TRANSLIT
  $converted = iconv($charset, 'UTF-8//TRANSLIT', $contents);
  if ($converted !== false) {
  return $converted;
  }
  } catch (IconvException $e) {
  try {
  // Затем с IGNORE
  $converted = iconv($charset, 'UTF-8//IGNORE', $contents);
  if ($converted !== false) {
  return $converted;
  }
  } catch (IconvException $e) {
  // Если iconv не сработал, пробуем альтернативные методы
  }
  }

  // Если ничего не помогло, пробуем определить кодировку и конвертировать
  $detectedEncoding = mb_detect_encoding($contents, ['UTF-8', 'Windows-1251', 'KOI8-R', 'ISO-8859-5'], true);
  if ($detectedEncoding && $detectedEncoding !== 'UTF-8') {
  $converted = mb_convert_encoding($contents, 'UTF-8', $detectedEncoding);
  if ($converted !== false) {
  return $converted;
  }
  }

  // Если все методы не сработали, возвращаем исходное содержимое
  return $contents;
  }

  return $contents;
  }

[trasher]

@Aleko-SaR your comment is not really lisible, please fix. There are many comments that are not in English, please translate.

For code changes, please open a PR. There is already one existing (#22998); but it does not work; therefore you can propose your own.