2.0.23

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@pushrocks/smartnginx",
-  "version": "2.0.16",
+  "version": "2.0.23",
   "lockfileVersion": 1,
   "requires": true,
   "dependencies": {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pushrocks/smartnginx",
-  "version": "2.0.16",
+  "version": "2.0.23",
   "private": false,
   "description": "control nginx from node, TypeScript ready",
   "main": "dist/index.js",

test/test.ts

@@ -40,6 +40,12 @@ tap.test('.deploy() should deploy a config from an instance', async () => {
   await testSmartNginx.deploy();
 });
 
+tap.test('should not redeploy', async () => {
+  testSmartNginx.addHostCandidate(testNginxZone01);
+  testSmartNginx.addHostCandidate(testNginxZone02);
+  await testSmartNginx.deploy();
+});
+
 tap.test('.stop() should end the process', async () => {
   testSmartNginx.nginxProcess.stop();
 });

ts/smartnginx.classes.smartnginx.ts

@@ -70,12 +70,12 @@ export class SmartNginx {
    * check wether there has been a diverging host configuration
    * this function will only redeploy the nginx configuration in case there has been a change
    */
-  private areHostsDiverged(): boolean {
+  private async areHostsDiverged(): Promise<boolean> {
     let hostCounter = 0;
     let unfoundHosts = 0;
-    this.hostCandidates.forEach(hostCandidateArg => {
+    await this.hostCandidates.forEach(async hostCandidateArg => {
       let foundHost = false;
-      this.deployedHosts.forEach(deployedHostArg => {
+      await this.deployedHosts.forEach(async deployedHostArg => {
         if (
           hostCandidateArg.hostName === deployedHostArg.hostName &&
           hostCandidateArg.destination === deployedHostArg.destination
@@ -89,7 +89,7 @@ export class SmartNginx {
       }
     });
     return (
-      this.deployedHosts.getArray.length !== this.hostCandidates.getArray().length ||
+      this.deployedHosts.getArray().length !== this.hostCandidates.getArray().length ||
       hostCounter !== this.deployedHosts.getArray().length ||
       unfoundHosts !== 0
     );
@@ -99,7 +99,7 @@ export class SmartNginx {
    * deploy the current stack and restart nginx
    */
   public async deploy() {
-    if (this.areHostsDiverged()) {
+    if (await this.areHostsDiverged()) {
       this.logger.log('ok', `hosts have diverged, trigger config deployment and nginx reload!`);
       this.deployedHosts.wipe();
       this.deployedHosts.addArray(this.hostCandidates.getArray());

ts/smartnginx.snippets.ts

@@ -74,10 +74,14 @@ export let getBaseConfigString = () => {
 export let getHostConfigString = (hostNameArg: string, destinationIpArg: string) => {
   let hostConfig = plugins.smartstring.indent.normalize(`
 		upstream ${hostNameArg} {
+			keepalive 100;
 			server ${destinationIpArg};
 		}
 
 		server {
+			# The keepalive parameter sets the maximum number of idle keepalive connections
+  		# to upstream servers that are preserved in the cache of each worker process. When
+			# this number is exceeded, the least recently used connections are closed.
 			listen *:80 ;
 			server_name ${hostNameArg};
 			rewrite        ^ https://${hostNameArg}$request_uri? permanent;
@@ -88,17 +92,14 @@ export let getHostConfigString = (hostNameArg: string, destinationIpArg: string)
 			server_name ${hostNameArg};
 			ssl_certificate ${paths.nginxHostDirPath}/${hostNameArg}.public.pem;
 			ssl_certificate_key ${paths.nginxHostDirPath}/${hostNameArg}.private.pem;
-
 			location / {
+				proxy_http_version 1.1;
 				proxy_set_header Host $http_host;
 				proxy_set_header X-Real-IP $remote_addr;
 				proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
 				proxy_set_header X-Forwarded-Proto $scheme;
 				proxy_pass http://${hostNameArg};
 			}
-			location ~ /\.git {
-				deny all;
-			}
 		}
 	`);
   return hostConfig;