Re: [PATCH common v2 1/2] fix #5978: pem parser: relax parsing of chain entries:

[Thomas Lamprecht <t.lamprecht@proxmox.com>]

Am 17.06.26 um 14:43 schrieb Thomas Ellmenreich:
> Relaxes the parser to allow for text and whitespaces inbetween certchain
> entries. The splitting of PEM chains was also reworked to split each entry at
> its end, grouping it with its leading text.
> 
> Added testsuite to cover a bunch of parsing edge cases.
> 
> Signed-off-by: Thomas Ellmenreich <t.ellmenreich@proxmox.com>
> ---
>  src/PVE/Certificate.pm |  25 +++-
>  test/Makefile          |   2 +
>  test/check_pem_test.pl | 332 +++++++++++++++++++++++++++++++++++++++++
>  test/split_pem_test.pl | 272 +++++++++++++++++++++++++++++++++
>  4 files changed, 625 insertions(+), 6 deletions(-)
>  create mode 100755 test/check_pem_test.pl
>  create mode 100755 test/split_pem_test.pl
> 
Noticed a few whitespace errors and formatting issues on applying.

Please ensure code is nicely formatted, for perl we use proxmox-perltidy, which
is a wrapper around upstream perltidy with a locked in configuration that matches
our style guide as closely as possible. Most repos provide a top level "tidy"
make target to simplify running this.