Improperly formatted armor header because Version header has no value

[camoredo]

Hi, would just like to ask if there is any way I can bypass the header check for empty value? I'm getting Error: Improperly formatted armor header: Version:\n because the header's value is undefined

For reference, here's the PGP message format:

-----BEGIN PGP MESSAGE-----
Version:

........
-----END PGP MESSAGE-----

Thank you

[twiss]

Hi 👋 For now there isn't. But fwiw, the Version: header is entirely inconsequential and unused, so as a workaround you can also just remove that line before parsing. We could also make the parsing more lenient, a PR would be welcome.

[camoredo]

Thanks for answering, @twiss! I'm planning to create a PR for it. Just to clarify the changes for it, should we allow no value for all headers or only if the header is Version?

[twiss]

For all headers is fine. Tbh, I would just change this line:

openpgpjs/src/encoding/armor.js

Line 193 in 93644b7

throw new Error('Improperly formatted armor header: ' + headers[i]);

into a debug error like the one below. Note that the guidance on this point changed between RFC4880 and the crypto refresh, so the comment above it can be updated as well to quote the latter instead. (Thanks in advance!)

[camoredo]

Got it, I've added a PR here #1557. Thanks!