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.

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

(MODULES-10831) key is expired if all subkeys are expired #1104

Merged
merged 1 commit into from
May 12, 2023

Conversation

kenyon
Copy link

@kenyon kenyon commented Apr 22, 2023

Previously, subkeys were not considered at all in the determination of whether a key was expired. Now this looks at all of the subkeys, and if they are all expired, considers the whole key expired.

@kenyon kenyon requested a review from a team as a code owner April 22, 2023 18:58
@CLAassistant
Copy link

CLAassistant commented Apr 22, 2023

CLA assistant check
All committers have signed the CLA.

@kenyon kenyon force-pushed the refresh-on-expired-subkeys branch from ba05c8f to f615010 Compare April 22, 2023 19:06
@kenyon kenyon marked this pull request as draft April 23, 2023 14:59
@kenyon kenyon force-pushed the refresh-on-expired-subkeys branch from f615010 to 42681d2 Compare April 24, 2023 22:31
@kenyon kenyon marked this pull request as ready for review April 24, 2023 22:46
@kenyon
Copy link
Author

kenyon commented Apr 24, 2023

This is ready.

@kenyon kenyon force-pushed the refresh-on-expired-subkeys branch 2 times, most recently from 949c77d to 99894a3 Compare April 29, 2023 20:06
@kenyon kenyon marked this pull request as draft May 3, 2023 22:31
@kenyon kenyon force-pushed the refresh-on-expired-subkeys branch from 99894a3 to 058e794 Compare May 3, 2023 22:38
Previously, subkeys were not considered at all in the determination of
whether a key was expired. Now this looks at all of the subkeys, and if
they are all expired, considers the whole key expired.
@kenyon kenyon force-pushed the refresh-on-expired-subkeys branch from 058e794 to cd31545 Compare May 3, 2023 22:51
@kenyon
Copy link
Author

kenyon commented May 3, 2023

Added tests, so now this is really ready. 😎

@kenyon kenyon marked this pull request as ready for review May 3, 2023 22:59
Copy link

@jordanbreen28 jordanbreen28 left a comment

Choose a reason for hiding this comment

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

This is great @kenyon.
LGTM

@jordanbreen28 jordanbreen28 merged commit c1c27eb into puppetlabs:main May 12, 2023
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