Merge pull request #806 from jc21/develop

v2.7.2
Version bump and acknowledgements
2021-01-07 09:35:35 +10:00 · 2021-01-06 21:48:10 +10:00 · 2021-01-06 13:14:54 +10:00 · 2021-01-05 18:47:22 +01:00 · 2021-01-05 18:41:26 +01:00 · 2021-01-05 18:35:18 +01:00
18 changed files with 109 additions and 43 deletions
--- a/.version
+++ b/.version
@ -1 +1 @@
-2.7.0
+2.7.2
--- a/README.md
+++ b/README.md
@ -1,7 +1,7 @@
 <p align="center">
 	<img src="https://nginxproxymanager.com/github.png">
 	<br><br>
-	<img src="https://img.shields.io/badge/version-2.7.0-green.svg?style=for-the-badge">
+	<img src="https://img.shields.io/badge/version-2.7.2-green.svg?style=for-the-badge">
 	<a href="https://hub.docker.com/repository/docker/jc21/nginx-proxy-manager">
 		<img src="https://img.shields.io/docker/stars/jc21/nginx-proxy-manager.svg?style=for-the-badge">
 	</a>
@ -205,6 +205,30 @@ Special thanks to the following contributors:
 				<br /><sub><b>Philip Mooney</b></sub>
 			</a>
 		</td>
+		<td align="center">
+			<a href="https://github.com/WaterCalm">
+				<img src="https://avatars1.githubusercontent.com/u/23502129?s=400&v=4" width="80px;" alt=""/>
+				<br /><sub><b>WaterCalm</b></sub>
+			</a>
+		</td>
+		<td align="center">
+			<a href="https://github.com/lebrou34">
+				<img src="https://avatars1.githubusercontent.com/u/16373103?s=460&v=4" width="80px;" alt=""/>
+				<br /><sub><b>lebrou34</b></sub>
+			</a>
+		</td>
+		<td align="center">
+			<a href="https://github.com/lightglitch">
+				<img src="https://avatars0.githubusercontent.com/u/196953?s=460&v=4" width="80px;" alt=""/>
+				<br /><sub><b>Mário Franco</b></sub>
+			</a>
+		</td>
+		<td align="center">
+			<a href="https://github.com/klutchell">
+				<img src="https://avatars3.githubusercontent.com/u/20458272?s=460&v=4" width="80px;" alt=""/>
+				<br /><sub><b>Kyle Harding</b></sub>
+			</a>
+		</td>
 	</tr>
 </table>
 <!-- markdownlint-enable -->
--- a/backend/index.js
+++ b/backend/index.js
@ -13,7 +13,6 @@ async function appStart () {
 	const internalCertificate = require('./internal/certificate');
 	const internalIpRanges    = require('./internal/ip_ranges');

-
 	return migrate.latest()
 		.then(setup)
 		.then(() => {
@ -43,13 +42,14 @@ async function appStart () {
 		});
 }

-async function createDbConfigFromEnvironment(){
+async function createDbConfigFromEnvironment() {
 	return new Promise((resolve, reject) => {
-		const envMysqlHost  = process.env.DB_MYSQL_HOST;
-		const envMysqlPort  = process.env.DB_MYSQL_PORT;
-		const envMysqlUser  = process.env.DB_MYSQL_USER;
-		const envMysqlName  = process.env.DB_MYSQL_NAME;
-		const envSqliteFile = process.env.DB_SQLITE_FILE;
+		const envMysqlHost  = process.env.DB_MYSQL_HOST || null;
+		const envMysqlPort  = process.env.DB_MYSQL_PORT || null;
+		const envMysqlUser  = process.env.DB_MYSQL_USER || null;
+		const envMysqlName  = process.env.DB_MYSQL_NAME || null;
+		const envSqliteFile = process.env.DB_SQLITE_FILE || null;
+
 		if ((envMysqlHost && envMysqlPort && envMysqlUser && envMysqlName) || envSqliteFile) {
 			const fs       = require('fs');
 			const filename = (process.env.NODE_CONFIG_DIR || './config') + '/' + (process.env.NODE_ENV || 'default') + '.json';
@ -119,7 +119,7 @@ async function createDbConfigFromEnvironment(){
 				}
 			});
 		} else {
-			// resolve();
+			resolve();
 		}
 	});
 }
@ -130,3 +130,4 @@ try {
 	logger.error(err.message, err);
 	process.exit(1);
 }
+
--- a/backend/internal/certificate.js
+++ b/backend/internal/certificate.js
@ -615,18 +615,26 @@ const internalCertificate = {
 	checkPrivateKey: (private_key) => {
 		return tempWrite(private_key, '/tmp')
 			.then((filepath) => {
-				let key_type = private_key.includes('-----BEGIN RSA') ? 'rsa' : 'ec';
-				return utils.exec('openssl ' + key_type + ' -in ' + filepath + ' -check -noout 2>&1 ')
-					.then((result) => {
-						if (!result.toLowerCase().includes('key ok') && !result.toLowerCase().includes('key valid') ) {
-							throw new error.ValidationError('Result Validation Error: ' + result);
-						}
-						fs.unlinkSync(filepath);
-						return true;
-					}).catch((err) => {
-						fs.unlinkSync(filepath);
-						throw new error.ValidationError('Certificate Key is not valid (' + err.message + ')', err);
-					});
+				return new Promise((resolve, reject) => {
+					const failTimeout = setTimeout(() => {
+						reject(new error.ValidationError('Result Validation Error: Validation timed out. This could be due to the key being passphrase-protected.'));
+					}, 10000);
+					utils
+						.exec('openssl pkey -in ' + filepath + ' -check -noout 2>&1 ')
+						.then((result) => {
+							clearTimeout(failTimeout);
+							if (!result.toLowerCase().includes('key is valid')) {
+								reject(new error.ValidationError('Result Validation Error: ' + result));
+							}
+							fs.unlinkSync(filepath);
+							resolve(true);
+						})
+						.catch((err) => {
+							clearTimeout(failTimeout);
+							fs.unlinkSync(filepath);
+							reject(new error.ValidationError('Certificate Key is not valid (' + err.message + ')', err));
+						});
+				});
 			});
 	},

--- a/backend/internal/host.js
+++ b/backend/internal/host.js
@ -106,7 +106,7 @@ const internalHost = {
 					response_object.total_count      += response_object.redirection_hosts.length;
 				}

-				if (promises_results[1]) {
+				if (promises_results[2]) {
 					// Dead Hosts
 					response_object.dead_hosts   = internalHost._getHostsWithDomains(promises_results[2], domain_names);
 					response_object.total_count += response_object.dead_hosts.length;
@ -158,7 +158,7 @@ const internalHost = {
 					}
 				}

-				if (promises_results[1]) {
+				if (promises_results[2]) {
 					// Dead Hosts
 					if (internalHost._checkHostnameRecordsTaken(hostname, promises_results[2], ignore_type === 'dead' && ignore_id ? ignore_id : 0)) {
 						is_taken = true;
--- a/backend/yarn.lock
+++ b/backend/yarn.lock
@ -1548,9 +1548,9 @@ inherits@2.0.3:
  integrity sha1-Yzwsg+PaQqUC9SRmAiSA9CCCYd4=

 ini@^1.3.4, ini@^1.3.5, ini@~1.3.0:
-  version "1.3.5"
-  resolved "https://registry.yarnpkg.com/ini/-/ini-1.3.5.tgz#eee25f56db1c9ec6085e0c22778083f596abf927"
-  integrity sha512-RZY5huIKCMRWDUqZlEi72f/lmXKMvuszcMBduliQ3nnWbx9X/ZBQO7DijMEYS9EhHBb2qacRUMtC7svLwe0lcw==
+  version "1.3.8"
+  resolved "https://registry.yarnpkg.com/ini/-/ini-1.3.8.tgz#a29da425b48806f34767a4efce397269af28432c"
+  integrity sha512-JV/yugV2uzW5iMRSiZAyDtQd+nxtUnjeLt0acNdw98kKLrvuRVyB80tsREOE7yvGVgalhZ6RNXCmEHkUKBKxew==

 inquirer@^7.0.0:
  version "7.3.3"
--- a/docker/Dockerfile
+++ b/docker/Dockerfile
@ -34,7 +34,7 @@ EXPOSE 443
 COPY docker/rootfs      /
 ADD backend             /app
 ADD frontend/dist       /app/frontend
-COPY global							/app/global
+COPY global             /app/global

 WORKDIR /app
 RUN yarn install
--- a/docker/docker-compose.ci.yml
+++ b/docker/docker-compose.ci.yml
@ -47,8 +47,8 @@ services:
  cypress-mysql:
    image: ${IMAGE}-cypress:ci-${BUILD_NUMBER}
    build:
-      context: ../
-      dockerfile: test/cypress/Dockerfile
+      context: ../test/
+      dockerfile: cypress/Dockerfile
    environment:
      CYPRESS_baseUrl: "http://fullstack-mysql:81"
    volumes:
@ -58,8 +58,8 @@ services:
  cypress-sqlite:
    image: ${IMAGE}-cypress:ci-${BUILD_NUMBER}
    build:
-      context: ../
-      dockerfile: test/cypress/Dockerfile
+      context: ../test/
+      dockerfile: cypress/Dockerfile
    environment:
      CYPRESS_baseUrl: "http://fullstack-sqlite:81"
    volumes:
--- a/docker/rootfs/etc/nginx/nginx.conf
+++ b/docker/rootfs/etc/nginx/nginx.conf
@ -84,6 +84,9 @@ http {
 stream {
 	# Files generated by NPM
 	include /data/nginx/stream/*.conf;
+
+	# Custom
+	include /data/nginx/custom/stream[.]conf;
 }

 # Custom
--- a/docs/advanced-config/README.md
+++ b/docs/advanced-config/README.md
@ -24,6 +24,7 @@ You can add your custom configuration snippet files at `/data/nginx/custom` as f

 - `/data/nginx/custom/root.conf`: Included at the very end of nginx.conf
 - `/data/nginx/custom/http.conf`: Included at the end of the main http block
+ - `/data/nginx/custom/stream.conf`: Included at the end of the main stream block
 - `/data/nginx/custom/server_proxy.conf`: Included at the end of every proxy server block
 - `/data/nginx/custom/server_redirect.conf`: Included at the end of every redirection server block
 - `/data/nginx/custom/server_stream.conf`: Included at the end of every stream server block
--- a/docs/yarn.lock
+++ b/docs/yarn.lock
@ -5125,9 +5125,9 @@ inherits@2.0.3:
  integrity sha1-Yzwsg+PaQqUC9SRmAiSA9CCCYd4=

 ini@^1.3.5, ini@~1.3.0:
-  version "1.3.5"
-  resolved "https://registry.yarnpkg.com/ini/-/ini-1.3.5.tgz#eee25f56db1c9ec6085e0c22778083f596abf927"
-  integrity sha512-RZY5huIKCMRWDUqZlEi72f/lmXKMvuszcMBduliQ3nnWbx9X/ZBQO7DijMEYS9EhHBb2qacRUMtC7svLwe0lcw==
+  version "1.3.8"
+  resolved "https://registry.yarnpkg.com/ini/-/ini-1.3.8.tgz#a29da425b48806f34767a4efce397269af28432c"
+  integrity sha512-JV/yugV2uzW5iMRSiZAyDtQd+nxtUnjeLt0acNdw98kKLrvuRVyB80tsREOE7yvGVgalhZ6RNXCmEHkUKBKxew==

 internal-ip@^4.3.0:
  version "4.3.0"
--- a/frontend/js/app/nginx/certificates/form.ejs
+++ b/frontend/js/app/nginx/certificates/form.ejs
@ -129,6 +129,9 @@
                    </div>
                <% } else if (provider === 'other') { %>
                    <!-- Other -->
+                    <div class="col-sm-12 col-md-12">
+                        <div class="text-blue mb-4"><i class="fe fe-alert-triangle"></i> <%= i18n('ssl', 'passphrase-protection-support-info') %></div>
+                    </div>
                    <div class="col-sm-12 col-md-12">
                        <div class="form-group">
                            <label class="form-label"><%- i18n('str', 'name') %> <span class="form-required">*</span></label>
--- a/frontend/js/i18n/messages.json
+++ b/frontend/js/i18n/messages.json
@ -112,7 +112,8 @@
      "stored-as-plaintext-info": "This data will be stored as plaintext in the database and in a file!",
      "propagation-seconds": "Propagation Seconds",
      "propagation-seconds-info": "Leave empty to use the plugins default value. Number of seconds to wait for DNS propagation.",
-      "processing-info": "Processing... This might take a few minutes."
+      "processing-info": "Processing... This might take a few minutes.",
+      "passphrase-protection-support-info": "Key files protected with a passphrase are not supported."
    },
    "proxy-hosts": {
      "title": "Proxy Hosts",
--- a/global/certbot-dns-plugins.js
+++ b/global/certbot-dns-plugins.js
@ -20,6 +20,16 @@
 */

 module.exports = {
+	aliyun: {
+		display_name:    'Aliyun',
+		package_name:    'certbot-dns-aliyun',
+		package_version: '0.38.1',
+		dependencies:    '',
+		credentials:     `certbot_dns_aliyun:dns_aliyun_access_key = 12345678
+certbot_dns_aliyun:dns_aliyun_access_key_secret = 1234567890abcdef1234567890abcdef`,
+		full_plugin_name: 'certbot-dns-aliyun:dns-aliyun',
+	},
+	//####################################################//
 	cloudflare: {
 		display_name:    'Cloudflare',
 		package_name:    'certbot-dns-cloudflare',
@ -110,6 +120,15 @@ certbot_dns_dnspod:dns_dnspod_api_token = "DNSPOD-API-TOKEN"`,
 		full_plugin_name: 'certbot-dns-dnspod:dns-dnspod',
 	},
 	//####################################################//
+	gandi: {
+		display_name:     'Gandi Live DNS',
+		package_name:     'certbot_plugin_gandi',
+		package_version:  '1.2.5',
+		dependencies:     '',
+		credentials:      'certbot_plugin_gandi:dns_api_key = APIKEY',
+		full_plugin_name: 'certbot-plugin-gandi:dns',
+	},
+	//####################################################//
 	google: {
 		display_name:    'Google',
 		package_name:    'certbot-dns-google',
@ -272,4 +291,4 @@ aws_secret_access_key=wJalrXUtnFEMI/K7MDENG/bPxRfiCYEXAMPLEKEY`,
 		credentials:      'certbot_dns_vultr:dns_vultr_key = YOUR_VULTR_API_KEY',
 		full_plugin_name: 'certbot-dns-vultr:dns-vultr',
 	},
-};
+};
--- a/test/.dockerignore
+++ b/test/.dockerignore
@ -0,0 +1 @@
+node_modules
--- a/test/cypress/Dockerfile
+++ b/test/cypress/Dockerfile
@ -1,6 +1,11 @@
-FROM cypress/included:4.12.1
+FROM cypress/included:5.6.0

-COPY --chown=1000 ./test /test
+COPY --chown=1000 ./ /test
+
+# mkcert
+ENV MKCERT=1.4.2
+RUN wget -O /usr/bin/mkcert "https://github.com/FiloSottile/mkcert/releases/download/v${MKCERT}/mkcert-v${MKCERT}-linux-amd64" \
+	&& chmod +x /usr/bin/mkcert

 WORKDIR /test
 RUN yarn install
--- a/test/package.json
+++ b/test/package.json
@ -7,7 +7,7 @@
 		"@jc21/cypress-swagger-validation": "^0.0.9",
 		"@jc21/restler": "^3.4.0",
 		"chalk": "^4.1.0",
-		"cypress": "^4.12.1",
+		"cypress": "^5.6.0",
 		"cypress-multi-reporters": "^1.4.0",
 		"cypress-plugin-retries": "^1.5.2",
 		"eslint": "^7.6.0",
--- a/test/yarn.lock
+++ b/test/yarn.lock
@ -1293,9 +1293,9 @@ inherits@2, inherits@^2.0.3, inherits@~2.0.3:
  integrity sha512-k/vGaX4/Yla3WzyMCvTQOXYeIHvqOKtnqBduzTHpzpQZzAskKMhZ2K+EnBiSM9zGSoIFeMpXKxa4dYeZIQqewQ==

 ini@^1.3.5:
-  version "1.3.5"
-  resolved "https://registry.yarnpkg.com/ini/-/ini-1.3.5.tgz#eee25f56db1c9ec6085e0c22778083f596abf927"
-  integrity sha512-RZY5huIKCMRWDUqZlEi72f/lmXKMvuszcMBduliQ3nnWbx9X/ZBQO7DijMEYS9EhHBb2qacRUMtC7svLwe0lcw==
+  version "1.3.8"
+  resolved "https://registry.yarnpkg.com/ini/-/ini-1.3.8.tgz#a29da425b48806f34767a4efce397269af28432c"
+  integrity sha512-JV/yugV2uzW5iMRSiZAyDtQd+nxtUnjeLt0acNdw98kKLrvuRVyB80tsREOE7yvGVgalhZ6RNXCmEHkUKBKxew==

 is-arguments@^1.0.4:
  version "1.0.4"
Author	SHA1	Message	Date
jc21	bfb328238e	Merge pull request #806 from jc21/develop v2.7.2	2021-01-07 09:35:35 +10:00
Jamie Curnow	64cc4f57d6	Version bump and acknowledgements	2021-01-06 21:48:10 +10:00
jc21	7a3c91c6a4	Merge pull request #804 from lebrou34/master Add Gandi Live DNS to certbot-dns-plugins.js	2021-01-06 13:14:54 +10:00
lebrou34	508bc62852	Update certbot-dns-plugins.js	2021-01-05 18:47:22 +01:00
lebrou34	59e8446d47	Update certbot-dns-plugins.js	2021-01-05 18:41:26 +01:00
lebrou34	d13596d2f7	Update certbot-dns-plugins.js	2021-01-05 18:35:18 +01:00
lebrou34	9adccfa341	Update certbot-dns-plugins.js	2021-01-05 18:19:27 +01:00
lebrou34	5cc3b53378	Update certbot-dns-plugins.js	2021-01-05 18:11:10 +01:00
lebrou34	b62b0a2fb7	Update certbot-dns-plugins.js	2021-01-05 17:36:47 +01:00
jc21	1faac4edf2	Merge pull request #750 from klutchell/klutchell-patch-1 allow custom stream conf	2021-01-03 20:14:55 +10:00
jc21	4c60dce169	Merge pull request #796 from lightglitch/patch-1 Fix dead hosts verification count	2021-01-03 20:14:36 +10:00
jc21	771f31f44d	Merge pull request #770 from jc21/dependabot/npm_and_yarn/backend/ini-1.3.8 Bump ini from 1.3.5 to 1.3.8 in /backend	2021-01-03 20:14:21 +10:00
jc21	8bedb95e1d	Merge pull request #771 from jc21/dependabot/npm_and_yarn/test/ini-1.3.8 Bump ini from 1.3.5 to 1.3.8 in /test	2021-01-03 20:14:03 +10:00
jc21	ac4be08df2	Merge pull request #772 from jc21/dependabot/npm_and_yarn/docs/ini-1.3.8 Bump ini from 1.3.5 to 1.3.8 in /docs	2021-01-03 20:13:49 +10:00
jc21	0d6e058e23	Merge pull request #774 from chaptergy/better-custom-certificate-handling Better custom certificate handling	2021-01-03 12:09:55 +10:00
Mário Franco	bee2ceb667	Fix dead hosts verification count	2021-01-02 04:58:14 +00:00
chaptergy	6af13d4f40	Removes explicit privkeytype check and adds passphrase error	2020-12-14 12:08:39 +01:00
dependabot[bot]	9dd0ebd899	Bump ini from 1.3.5 to 1.3.8 in /docs Bumps [ini](https://github.com/isaacs/ini) from 1.3.5 to 1.3.8. - [Release notes](https://github.com/isaacs/ini/releases) - [Commits](https://github.com/isaacs/ini/compare/v1.3.5...v1.3.8) Signed-off-by: dependabot[bot] <support@github.com>	2020-12-12 10:17:11 +00:00
dependabot[bot]	6e97bfa717	Bump ini from 1.3.5 to 1.3.8 in /test Bumps [ini](https://github.com/isaacs/ini) from 1.3.5 to 1.3.8. - [Release notes](https://github.com/isaacs/ini/releases) - [Commits](https://github.com/isaacs/ini/compare/v1.3.5...v1.3.8) Signed-off-by: dependabot[bot] <support@github.com>	2020-12-12 08:23:59 +00:00
dependabot[bot]	07b69f41eb	Bump ini from 1.3.5 to 1.3.8 in /backend Bumps [ini](https://github.com/isaacs/ini) from 1.3.5 to 1.3.8. - [Release notes](https://github.com/isaacs/ini/releases) - [Commits](https://github.com/isaacs/ini/compare/v1.3.5...v1.3.8) Signed-off-by: dependabot[bot] <support@github.com>	2020-12-12 08:23:45 +00:00
Kyle Harding	6bd2ac7d6d	Update README.md	2020-12-01 14:24:14 -05:00
Kyle Harding	528e5ef3bc	allow custom stream conf Allow a top-level custom `stream` configuration file to be loaded.	2020-12-01 14:22:31 -05:00
Jamie Curnow	bc1c50ac92	Added contributor	2020-11-22 21:50:57 +10:00
jc21	8c2ab42b75	Merge pull request #738 from WaterCalm/master add aliyun DNS plugin	2020-11-22 21:48:24 +10:00
WaterCalm	62053d15d4	add aliyun DNS plugin add aliyun DNS plugin	2020-11-22 16:08:56 +08:00
Jamie Curnow	6fed642aba	Cypress docker build should be faster and added mkcert for later	2020-11-22 16:57:12 +10:00
jc21	72ac549a58	Merge pull request #729 from jc21/develop v2.7.1	2020-11-18 22:55:47 +10:00
jc21	9f38617135	Merge branch 'master' into develop	2020-11-18 22:55:37 +10:00
Jamie Curnow	94eec805df	Version bump	2020-11-18 21:46:21 +10:00
Jamie Curnow	05a940e732	Fix instances where config file exists and env vars don't	2020-11-18 21:42:03 +10:00
 @ -1 +1 @@
 .7.0
 .7.2