Skip to content

Commit e0ab70b

Browse files
committed
fix: ensure organizationalUnit is converted before engine initialization
The migration was failing during upgrade to stable32 because it tried to set organizationalUnit as a string when the property expects an array. The issue occurred because addConfigPathToOpenSsl() (which calls getEngine() and triggers populateInstance()) was executed before convertRootCertOuStringToArray(). This fix reorders the method calls in preSchemaChange() to convert the OU value to an array before initializing the engine, preventing the TypeError. Fixes the error: TypeError: Cannot assign string to property OCA\Libresign\Handler\CertificateEngine\AEngineHandler::$organizationalUnit of type array Signed-off-by: Vitor Mattos <[email protected]>
1 parent c2a0ae9 commit e0ab70b

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

lib/Migration/Version13000Date20251031165700.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,8 +52,8 @@ public function __construct(
5252
*/
5353
#[Override]
5454
public function preSchemaChange(IOutput $output, Closure $schemaClosure, array $options): void {
55-
$this->addConfigPathToOpenSsl();
5655
$this->convertRootCertOuStringToArray();
56+
$this->addConfigPathToOpenSsl();
5757
$this->backupCrlDataToDisk();
5858
}
5959

0 commit comments

Comments
 (0)