fix(nginx): known model shouldn't be refreshed if Nginx reload failed.

Matt Tucker · Matt Tucker · commit 5efc2480608b · 2016-10-07T10:54:19.000-06:00
diff --git a/router.go b/router.go
@@ -50,7 +50,11 @@ func main() {
 			log.Printf("Failed to write new nginx configuration; continuing with existing configuration: %v", err)
 			continue
 		}
-		nginx.Reload()
+		err = nginx.Reload()
+		if err != nil {
+			log.Printf("Failed to reload nginx; continuing with existing configuration: %v", err)
+			continue
+		}
 		known = routerConfig
 	}
 }

Original file line number	Diff line number	Diff line change
`@@ -50,7 +50,11 @@ func main() {`
`50`	`50`	`log.Printf("Failed to write new nginx configuration; continuing with existing configuration: %v", err)`
`51`	`51`	`continue`
`52`	`52`	`}`
`53`		`- nginx.Reload()`
	`53`	`+ err = nginx.Reload()`
	`54`	`+ if err != nil {`
	`55`	`+ log.Printf("Failed to reload nginx; continuing with existing configuration: %v", err)`
	`56`	`+ continue`
	`57`	`+ }`
`54`	`58`	`known = routerConfig`
`55`	`59`	`}`
`56`	`60`	`}`