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

Let's Encrypt certificates are issued by R3, R4, E1 or E2 now #382

Merged
merged 1 commit into from
Feb 15, 2021

Conversation

jaykay-design
Copy link
Contributor

@automationsynd
Copy link

automationsynd commented Feb 11, 2021

@sjkp When will this PR be merged? I have certificates within the default 22 days before expiration that are not getting renewed because the current cert is issued by "R3" and the renew job is still looking for one containing "Let's Encrypt".

@automationsynd
Copy link

@jaykay-design any idea how soon this will be merged?

@jaykay-design
Copy link
Contributor Author

@automationsynd, nope. Not my repository so I can't do anything.

You can use the release I did on my PR branch if you want.

@joelnotified
Copy link

@jaykay-design looks like I've run into the same issue. All of my certs have stopped renewing, so thanks for this PR! Let's hope it gets merged and released soon.

In the meantime, do you know how to install an App Service extension from your zip release? Is this the way (manually adding it to a folder in Kudu) https://thefreezeteam.com/manually-add-extension-in-azure-app-service/?

@jaykay-design
Copy link
Contributor Author

In the meantime, do you know how to install an App Service extension from your zip release? Is this the way (manually adding it to a folder in Kudu) https://thefreezeteam.com/manually-add-extension-in-azure-app-service/?

The ZIP release only contains the source code of the project and not any executable so before being able to update the SiteExtension you would have to build the project first. Let me know if you need any help with that.

Once you have built the project you can just replace the files of the SiteExtensions similar as described in the link you provided.

@joelnotified
Copy link

@jaykay-design Alright! Thanks

var expiringCerts = certs.Where(s => s.ExpirationDate < DateTime.UtcNow.AddDays(renewXNumberOfDaysBeforeExpiration) && (s.Issuer.Contains("Let's Encrypt") || s.Issuer.Contains("Fake LE")));

var issuerNames = new[] { "Fake LE", "Let's Encrypt", "R3", "R4", "E1", "E2" };
var letsEncryptCerts = ExtractCertificates(body).Where(s => issuerNames.Any(i => s.Issuer.StartsWith(i, StringComparison.InvariantCulture)));

Choose a reason for hiding this comment

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

Just wondering about this change here @jaykay-design . letsEncryptCerts is never used right?

I guess it doesn't really matter, but just wanted to let you know.

@jaykay-design
Copy link
Contributor Author

Yes, you are right. Sorry for that.

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.

None yet

4 participants