[pve-devel] [PATCH installer v2 06/17] common: setup: deserialize `secure_boot` property from runtime env

[Christoph Heiss]

On Tue, Jul 23, 2024 at 04:31:35PM GMT, Aaron Lauterer wrote:
> In my tests, with secure boot disabled, it failed to parse the
> run-env-info.json because the Perl code wrote it this way:
>
> "secure_boot":""
>
> And it currently cannot parse a string. Setting it manually to:
>
> "secure_boot":0
>
> helped. The question is, if we want the parser to be more flexible or fix
> the Perl code that dumps that info.

FWIW: Testing this, I couldn't actually reproduce this.

Re-reading the dumping code in Proxmox/Install/RunEnv.pm, I'd also
expect this. In the non-secureboot case, the hash member isn't set, so
the key should not be dumped at all. Otherwise, 1 as value gets dumped.

So I'm not sure how the perl code would even get to dumping an empty
string ..