Skip to content

Add parameter SMIME::NoDefaultCA and remove unnecessary parameter#681

Open
pboguslawski wants to merge 4 commits intoznuny:rel-6_5-devfrom
ibpl:rel-6_5-IB#1161846
Open

Add parameter SMIME::NoDefaultCA and remove unnecessary parameter#681
pboguslawski wants to merge 4 commits intoznuny:rel-6_5-devfrom
ibpl:rel-6_5-IB#1161846

Conversation

@pboguslawski
Copy link
Copy Markdown
Contributor

Proposed change

Znuny does not disable OS default CA certificate sources on S/MIME verification which may lead to different verification results in different environments with same certs in SMIME::CertPath. For example: signature not trusted in Debian 11 may be trusted after upgrade to Debian 12 where default trust was modified.

This mod adds SMIME::NoDefaultCA (disabled by default for backward compatibility) to allow one to skip default OS CAs on S/MIME verification (and use trust only CAs in SMIME::CertPath like in Debian 11).

This mod also removes unnecessary parameter from
openssl smime -verify command.

Type of change

  • '1 - 🐞 bug 🐞' - Bugfix (non-breaking change which fixes an issue)

Additional information

Related: https://docs.openssl.org/master/man1/openssl-verification-options/#trusted-certificate-options
Author-Change-Id: IB#1161846

Checklist

  • The code change is tested and works locally.(❗)
  • There is no commented out code in this PR.(❕)
  • You improved or added new unit tests.(❕)
  • Local ZnunyCodePolicy passed.(❕)
  • Local UnitTests / Selenium passed.(❕)
  • GitHub workflow CI (UnitTests / Selenium) passed.(❗)

Znuny does not disable OS default CA certificate sources on S/MIME
verification which may lead to different verification results
in different environments with same certs in `SMIME::CertPath`.
For example: signature not trusted in Debian 11 may be trusted
after upgrade to Debian 12 where default trust was modified:

https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=805646

This mod adds `SMIME::NoDefaultCA` (disabled by default for backward
compatibility) to allow one to skip default OS CAs on S/MIME
verification (and use trust only CAs in `SMIME::CertPath` like
in Debian 11).

This mod also removes unnecessary parameter from
`openssl smime -verify`  command.

Related: https://docs.openssl.org/master/man1/openssl-verification-options/#trusted-certificate-options
Author-Change-Id: IB#1161846
Signed-off-by: Pawel Boguslawski <pawel.boguslawski@ib.pl>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant