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

Add ct_get_certificate_timestamp and ct_get_certificate_age_s functions #245

Merged
merged 1 commit into from
Jan 2, 2024

Conversation

hhorak
Copy link
Member

@hhorak hhorak commented Mar 29, 2022

These functions are expected to be used for checking whether a certificate was just
generated during the container assembing (e.g age of the certificate is less than a minute).

The certificates generation on assembling the container with the application is important
for not having the container images with certificates being part of the image.

@hhorak
Copy link
Member Author

hhorak commented Apr 26, 2022

[test]

@hhorak hhorak force-pushed the certificate_age branch 2 times, most recently from 488a037 to b7734af Compare May 12, 2022 06:36
@hhorak hhorak force-pushed the certificate_age branch from f4a2159 to 3cedf65 Compare May 12, 2022 19:50
@hhorak
Copy link
Member Author

hhorak commented May 12, 2022

[test]

1 similar comment
@hhorak
Copy link
Member Author

hhorak commented May 12, 2022

[test]

@hhorak
Copy link
Member Author

hhorak commented May 13, 2022

[test]

1 similar comment
@hhorak
Copy link
Member Author

hhorak commented May 13, 2022

[test]

@hhorak hhorak force-pushed the certificate_age branch from 3cedf65 to d952e33 Compare May 16, 2022 07:48
@hhorak hhorak force-pushed the certificate_age branch from d952e33 to a1562e9 Compare May 26, 2022 07:54
@hhorak
Copy link
Member Author

hhorak commented May 26, 2022

[test]

Copy link
Contributor

@zmiklank zmiklank left a comment

Choose a reason for hiding this comment

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

I am not sure, if this PR is still in queue to be reviewed, nonetheless, here is my review :).

I would also suggest squashing the commits before merging/rebasing.

Else LGTM.

test-lib.sh Outdated Show resolved Hide resolved
test-lib.sh Outdated Show resolved Hide resolved
test-lib.sh Outdated Show resolved Hide resolved
@pkubatrh
Copy link
Member

pkubatrh commented Aug 2, 2023

@hhorak Can you take a look at the comments?

@hhorak hhorak marked this pull request as draft December 15, 2023 10:37
test-lib.sh Fixed Show fixed Hide fixed
@hhorak
Copy link
Member Author

hhorak commented Dec 15, 2023

This PR is tested in sclorg/httpd-container#140, so not intended to be merged before it's working in CI in that PR.

…t_image_age_s functions

These functions are expected to be used for checking whether a certificate was just
generated during the container assembing (e.g age of the certificate is less than a minute).

The certificates generation on assembling the container with the application is important
for not having the container images with certificates being part of the image.
@hhorak hhorak marked this pull request as ready for review December 15, 2023 15:04
@hhorak
Copy link
Member Author

hhorak commented Dec 15, 2023

[test]

@hhorak
Copy link
Member Author

hhorak commented Dec 15, 2023

Rebased and sclorg/httpd-container#140 testing it right now, let's see whether tests there are green.

Copy link
Contributor

@zmiklank zmiklank left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for the great comments in the code!

@zmiklank zmiklank merged commit 1e82b93 into master Jan 2, 2024
27 of 28 checks passed
@zmiklank zmiklank deleted the certificate_age branch January 2, 2024 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants