From e80bb9c09f8c2dab1d174f75f4757bc517dd6854 Mon Sep 17 00:00:00 2001 From: Vincent Petry Date: Mon, 26 Oct 2020 11:51:37 +0100 Subject: [PATCH] Clarify import cert command Add hint that the certificate to be imported must be in PEM format (CRT does not parse). Improved error message to mention that the given path must be accessible by the web server user. Signed-off-by: Vincent Petry --- core/Command/Security/ImportCertificate.php | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/core/Command/Security/ImportCertificate.php b/core/Command/Security/ImportCertificate.php index 836f1e8b9d..e134f0f589 100644 --- a/core/Command/Security/ImportCertificate.php +++ b/core/Command/Security/ImportCertificate.php @@ -42,11 +42,11 @@ class ImportCertificate extends Base { protected function configure() { $this ->setName('security:certificates:import') - ->setDescription('import trusted certificate') + ->setDescription('import trusted certificate in PEM format') ->addArgument( 'path', InputArgument::REQUIRED, - 'path to the certificate to import' + 'path to the PEM certificate to import' ); } @@ -54,7 +54,7 @@ class ImportCertificate extends Base { $path = $input->getArgument('path'); if (!file_exists($path)) { - $output->writeln('certificate not found'); + $output->writeln('Certificate not found, please provide a path accessible by the web server user'); return 1; }