Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

TemporarySecureMimeContext never looks at certificate alternative subject for email when signing/encrypting #323

Closed
logicmediadk opened this issue Jul 31, 2017 · 5 comments
Closed

TemporarySecureMimeContext never looks at certificate alternative subject for email when signing/encrypting #323

logicmediadk opened this issue Jul 31, 2017 · 5 comments
Labels
bug Something isn't working

Comments

@logicmediadk
Copy link

When using MailboxAddress along with the TemporarySecureMimeContext the code only looks for the email address of the certificate in the subject. The code should also look at the alternative subject for the Rfc822Name (See https://github.com/bcgit/bc-csharp/blob/b19e68a517e56ef08cd2e50df4dcb8a96ddbe507/crypto/src/x509/extension/X509ExtensionUtil.cs#L65)
@logicmediadk logicmediadk changed the title TemporarySecureMimeContext only looks at certificate subject for email while ignoring alternative subject TemporarySecureMimeContext never looks at alternative subject for email Jul 31, 2017
@logicmediadk logicmediadk changed the title TemporarySecureMimeContext never looks at alternative subject for email TemporarySecureMimeContext never looks at certificate alternative subject for email when signing/encrypting Jul 31, 2017
jstedfast added a commit that referenced this issue Jul 31, 2017
@jstedfast
Fall back to getting the Subject Alternative Rfc822 Name if the Subje…
09ed900 
…ctEmailAddress fails

Fixes issue #323
@jstedfast jstedfast added the bug Something isn't working label Jul 31, 2017
@logicmediadk
Copy link
Author

It works, thanks for the fast fix!

@logicmediadk
Copy link
Author

Do you have a release plan for version 1.16.3?

@Nikkelmann
Copy link

Merged my account, hence the ghost.
Sorry for the notification.

@jstedfast
Copy link
Owner

I can try to release this weekend.

@Nikkelmann
Copy link

Sounds good, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jstedfast @Nikkelmann