Skip to content

Conversation

@dtomcej
Copy link
Contributor

@dtomcej dtomcej commented Jul 23, 2019

What does this PR do?

Adds missing KeyUsages for the default generated certificate

Motivation

Fixes #5006

Additional Notes

Chrome Pre-v75 did not strictly enforce the keyUsage parameter for self-signed certificates, as the user already had to allow them.

I have added in:

  • Digital Signature (Required by chrome)
  • Key Agreement (can be used with DH ciphers)
  • Data Encipherment (for encrypting data)

More information can be found in:
(https://www.ibm.com/support/knowledgecenter/sl/SSKTMJ_9.0.1/admin/conf_keyusageextensionsandextendedkeyusage_r.html)
and (https://www.ietf.org/rfc/rfc3280.txt) section 4.2.1.3 Key Usage

@traefiker traefiker added this to the 2.0 milestone Jul 23, 2019
@dtomcej dtomcej requested review from a team as code owners July 23, 2019 17:34
@dtomcej dtomcej changed the base branch from v2.0 to v1.7 July 23, 2019 17:35
@dtomcej dtomcej removed request for a team July 23, 2019 21:29
@ldez ldez modified the milestones: 2.0, 1.7 Jul 24, 2019
Copy link
Collaborator

@SantoDE SantoDE left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM :)

Copy link
Contributor

@dduportal dduportal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

Copy link
Contributor

@ldez ldez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants