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

Rewrite renew #1195

Merged
merged 6 commits into from
Jul 31, 2024
Merged

Rewrite renew #1195

merged 6 commits into from
Jul 31, 2024

Conversation

TinCanTech
Copy link
Collaborator

No description provided.

Add specific error message for 'renew'.

Re-insert 'renew_restore_move', to undo changes made by 'renew'.

Signed-off-by: Richard T Bonhomme <tincantech@protonmail.com>
To 'renew' a certificate, there are pre-requisites that must be met.

Attribuyes from the original certificate must match the new certificate.
The attributes supported by EasyRSA are now all correctly 'renewed'.

Signed-off-by: Richard T Bonhomme <tincantech@protonmail.com>
@TinCanTech TinCanTech self-assigned this Jul 28, 2024
@TinCanTech TinCanTech added the development Possible changes label Jul 28, 2024
@TinCanTech
Copy link
Collaborator Author

Updated UT includes test for renew and revoke-renewed.

Signed-off-by: Richard T Bonhomme <tincantech@protonmail.com>
@TinCanTech
Copy link
Collaborator Author

TinCanTech commented Jul 29, 2024

Also requires that EASYRSA_CP_EXT and EASYRSA_AUTO_SAN be disabled.

Netscape support is dropped by renew, unless explicitly requested.

Signed-off-by: Richard T Bonhomme <tincantech@protonmail.com>
Signed-off-by: Richard T Bonhomme <tincantech@protonmail.com>
Signed-off-by: Richard T Bonhomme <tincantech@protonmail.com>
@TinCanTech TinCanTech added this to the v3.2.1 milestone Jul 31, 2024
@TinCanTech TinCanTech removed the development Possible changes label Jul 31, 2024
@TinCanTech TinCanTech merged commit b812dbd into OpenVPN:master Jul 31, 2024
4 checks passed
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.

Consider re-adding renew for v3.2.1 - Only v3.2.0 does not have renew
1 participant