Skip to content
This repository was archived by the owner on Jan 3, 2025. It is now read-only.

Update 30-keygen #51

Closed
wants to merge 1 commit into from

Conversation

symetryczny
Copy link

linuxserver.io


  • I have read the contributing guideline and understand that I have made the correct modifications

Description:

A new certificate if generated if one is missing. I added a check to renew certificate if the one that already exists is close to expiring (expires within 30 days)

Benefits of this PR and context:

Keeps the certificate up to date

How Has This Been Tested?

Ran the script with no certificate present - a new one is generated
Ran the script when existing certificate is expired - a new one is generated
Ran the script when existing certificate is expiring within 365 days - no new certificate is generated

Source / References:

Add a check to renew certificate if is close to expiring (expires within 30 days)
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Thanks for opening this pull request! Be sure to follow the pull request template!

@github-actions
Copy link

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@symetryczny
Copy link
Author

What do you guys think about that?

@symetryczny symetryczny reopened this Mar 12, 2022
@github-actions
Copy link

github-actions bot commented May 7, 2022

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot closed this Apr 2, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 2, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants