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 command for testing which certificates are eligible for renewal #555

Merged
merged 4 commits into from May 10, 2022

Conversation

AndersBlomdell
Copy link
Contributor

Since it looks like easy-rsa is intended for automation (assumption based on -sbatch option), the possibilty to test / list which certificates are eligible for renewal might come in handy.

Re-submitted due to shellcheck errors in CI (deservedly bashed for using bash)

@TinCanTech
Copy link
Collaborator

TinCanTech commented May 4, 2022

The reason the test fails is due to a limitation of date.exe.

However, Windows also does not have find.exe ..

It's a good idea but not executed the way you are doing. It is preferable to parse the SSL database for names.

@AndersBlomdell
Copy link
Contributor Author

Parsing index.txt for finding candidates, next step is maybe to filter eligible dates in this step as well

@ecrist
Copy link
Member

ecrist commented May 5, 2022 via email

@TinCanTech TinCanTech self-assigned this May 5, 2022
@TinCanTech TinCanTech added this to the v3.1.1 milestone May 5, 2022
@TinCanTech
Copy link
Collaborator

TinCanTech commented May 5, 2022

@AndersBlomdell FTR: Easy-TLS already does something similar:

easytls status

This may fit in better there

Go crazy! ;-)

@AndersBlomdell
Copy link
Contributor Author

@AndersBlomdell FTR: Easy-TLS already does something similar:

easytls status

Since easytls is not part of my preferred distribution (Fedora), I'll leave it here (or make a small local script for my needs)

@TinCanTech
Copy link
Collaborator

TinCanTech commented May 7, 2022

Competing code: #557

I'm still happy to merge this one #555 ;-)

@TinCanTech TinCanTech added Full-Approval Merge is imminent documentation and removed initial-approval development Possible changes labels May 7, 2022
@TinCanTech TinCanTech modified the milestones: v3.1.1, v3.1.0 May 7, 2022
@TinCanTech TinCanTech merged commit 93bf14f into OpenVPN:master May 10, 2022
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.

None yet

3 participants