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

Allow --fix-offset to create post-dated certificates #804

Merged
merged 1 commit into from Dec 22, 2022

Conversation

TinCanTech
Copy link
Collaborator

Currently, --fix-offset does not allow creating post-dated cerificates. Instead, it rolls the start date back by one year.

The reason for this behavior is that --fix-offest was only meant to fix a day of the year but not which year.

This patch simply removes the check for a post-dated certificate, which allows not-before-date to be set upto one year in the future.

Signed-off-by: Richard T Bonhomme tincantech@protonmail.com

Currently, --fix-offset does not allow creating post-dated cerificates.
Instead, it rolls the start date back by one year.

The reason for this behavior is that --fix-offest was only meant to fix
a day of the year but not which year.

This patch simply removes the check for a post-dated certificate, which
allows `not-before-date` to be set upto one year in the future.

Signed-off-by: Richard T Bonhomme <tincantech@protonmail.com>
@TinCanTech TinCanTech self-assigned this Dec 7, 2022
@TinCanTech TinCanTech added this to the v3.1.2 - Probably milestone Dec 7, 2022
@TinCanTech TinCanTech linked an issue Dec 7, 2022 that may be closed by this pull request
@TinCanTech TinCanTech merged commit 97cea41 into OpenVPN:master Dec 22, 2022
@TinCanTech TinCanTech deleted the allow-post-date-fix-offset branch January 3, 2023 22:17
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.

--fix-offset behaviour is incorrect
1 participant