cleanupCerts: add progress

This commit is contained in:
Girish Ramakrishnan
2022-07-13 10:49:08 +05:30
parent 1549f6a4d0
commit 2af29fd844
2 changed files with 7 additions and 7 deletions
+6 -5
View File
@@ -664,14 +664,15 @@ async function renewCerts(options, auditSource, progressCallback) {
}
}
async function cleanupCerts(auditSource) {
async function cleanupCerts(auditSource, progressCallback) {
assert.strictEqual(typeof auditSource, 'object');
assert.strictEqual(typeof progressCallback, 'function');
const filenames = await fs.promises.readdir(paths.NGINX_CERT_DIR);
const certFilenames = filenames.filter(f => f.endsWith('.cert'));
const now = new Date();
debug('cleanupCerts: start');
progressCallback({ message: 'Checking expired certs for removal' });
const fqdns = [];
@@ -682,7 +683,7 @@ async function cleanupCerts(auditSource) {
if (now - notAfter >= (60 * 60 * 24 * 30 * 6 * 1000)) { // expired 6 months ago
const fqdn = certFilename.replace(/\.cert$/, '');
debug(`cleanupCerts: deleting certs of ${fqdn}`);
progressCallback({ message: `deleting certs of ${fqdn}` });
// it is safe to delete the certs of stopped apps because their nginx configs are removed
safe.fs.unlinkSync(certFilePath);
@@ -708,7 +709,7 @@ async function checkCerts(options, auditSource, progressCallback) {
assert.strictEqual(typeof progressCallback, 'function');
await renewCerts(options, auditSource, progressCallback);
await cleanupCerts(auditSource);
await cleanupCerts(auditSource, progressCallback);
}
function removeAppConfigs() {
@@ -717,7 +718,7 @@ function removeAppConfigs() {
debug('removeAppConfigs: reomving nginx configs of apps');
// remove all configs which are not the default or current dashboard
for (let appConfigFile of fs.readdirSync(paths.NGINX_APPCONFIG_DIR)) {
for (const appConfigFile of fs.readdirSync(paths.NGINX_APPCONFIG_DIR)) {
if (appConfigFile !== constants.NGINX_DEFAULT_CONFIG_FILE_NAME && appConfigFile !== dashboardConfigFilename) {
fs.unlinkSync(path.join(paths.NGINX_APPCONFIG_DIR, appConfigFile));
}