Merge pull request #7151 from LibreSign/backport/7149/stable33

[stable33] fix: signer common name array

Author: vitormattos

lib/Service/SignFileService.php

@@ -927,16 +927,33 @@ protected function getSignatureParams(): array {
 	}
 
 	private function buildBaseSignatureParams(array $certificateData): array {
+		$issuerCommonName = $this->normalizeCertificateFieldToString($certificateData['issuer']['CN'] ?? '');
+		$signerCommonName = $this->normalizeCertificateFieldToString($certificateData['subject']['CN'] ?? '');
+
 		return [
 			'DocumentUUID' => $this->libreSignFile?->getUuid(),
-			'IssuerCommonName' => $certificateData['issuer']['CN'] ?? '',
-			'SignerCommonName' => $certificateData['subject']['CN'] ?? '',
+			'IssuerCommonName' => $issuerCommonName,
+			'SignerCommonName' => $signerCommonName,
 			'LocalSignerTimezone' => $this->dateTimeZone->getTimeZone()->getName(),
 			'LocalSignerSignatureDateTime' => (new DateTime('now', new \DateTimeZone('UTC')))
 				->format(DateTimeInterface::ATOM)
 		];
 	}
 
+	private function normalizeCertificateFieldToString(mixed $value): string {
+		if (is_array($value)) {
+			$flattened = [];
+			array_walk_recursive($value, static function (mixed $item) use (&$flattened): void {
+				if ($item !== null) {
+					$flattened[] = (string)$item;
+				}
+			});
+			return implode(', ', $flattened);
+		}
+
+		return $value === null ? '' : (string)$value;
+	}
+
 	private function addEmailToSignatureParams(array $signatureParams, array $certificateData): array {
 		$email = $this->subjectAlternativeNameService->extractEmailFromCertificate($certificateData);
 		if ($email) {

tests/php/Unit/Service/SignFileServiceTest.php

@@ -1166,6 +1166,14 @@ public static function providerGetSignatureParamsCommonName(): array {
 				'LibreCode',
 				'Jane Doe',
 			],
+			'array CNs should be normalized' => [
+				[
+					'issuer' => ['CN' => ['LibreCode', 'CA']],
+					'subject' => ['CN' => ['Doe', 'Jane']],
+				],
+				'LibreCode, CA',
+				'Doe, Jane',
+			],
 			'empty CNs' => [
 				[
 					'issuer' => ['CN' => ''],