From 568dc907693f92c14df4f10db41d30edb338d96a Mon Sep 17 00:00:00 2001
From: arkpar <arkady.paronyan@gmail.com>
Date: Fri, 24 Jun 2016 14:20:39 +0200
Subject: [PATCH] Signer enabled by default for UI

---
 parity/cli.rs           | 5 ++++-
 parity/configuration.rs | 5 +++++
 parity/main.rs          | 4 ++--
 3 files changed, 11 insertions(+), 3 deletions(-)

diff --git a/parity/cli.rs b/parity/cli.rs
index 8a4e050af..0133ca0fb 100644
--- a/parity/cli.rs
+++ b/parity/cli.rs
@@ -117,7 +117,9 @@ API and Console Options:
                            [default: $HOME/.parity/dapps]
 
   --signer                 Enable Trusted Signer WebSocket endpoint used by
-                           Signer UIs.
+                           Signer UIs. Default if no command is specified.
+  --no-signer              Disable Trusted Signer WebSocket endpoint used by
+                           Signer UIs. Default if run with ui command.
   --signer-port PORT       Specify the port of Trusted Signer server
                            [default: 8180].
   --signer-path PATH       Specify directory where Signer UIs tokens should
@@ -280,6 +282,7 @@ pub struct Args {
 	pub flag_dapps_pass: Option<String>,
 	pub flag_dapps_path: String,
 	pub flag_signer: bool,
+	pub flag_no_signer: bool,
 	pub flag_signer_port: u16,
 	pub flag_signer_path: String,
 	pub flag_no_token: bool,
diff --git a/parity/configuration.rs b/parity/configuration.rs
index f0cbebdb4..925d2041c 100644
--- a/parity/configuration.rs
+++ b/parity/configuration.rs
@@ -467,6 +467,11 @@ impl Configuration {
 	pub fn dapps_enabled(&self) -> bool {
 		!self.args.flag_dapps_off && !self.args.flag_no_dapps
 	}
+
+	pub fn signer_enabled(&self) -> bool {
+		(self.args.cmd_ui && !self.args.flag_no_signer) ||
+		(!self.args.cmd_ui && self.args.flag_signer)
+	}
 }
 
 #[cfg(test)]
diff --git a/parity/main.rs b/parity/main.rs
index fd7e8a437..84f144396 100644
--- a/parity/main.rs
+++ b/parity/main.rs
@@ -192,14 +192,14 @@ fn execute_client(conf: Configuration, spec: Spec, client_config: ClientConfig)
 	let sync_config = conf.sync_config(&spec);
 
 	// Create and display a new token for UIs.
-	if conf.args.flag_signer && !conf.args.flag_no_token {
+	if conf.signer_enabled() && !conf.args.flag_no_token {
 		new_token(conf.directories().signer).unwrap_or_else(|e| {
 			die!("Error generating token: {:?}", e)
 		});
 	}
 
 	// Display warning about using unlock with signer
-	if conf.args.flag_signer && conf.args.flag_unlock.is_some() {
+	if conf.signer_enabled() && conf.args.flag_unlock.is_some() {
 		warn!("Using Trusted Signer and --unlock is not recommended!");
 		warn!("NOTE that Signer will not ask you to confirm transactions from unlocked account.");
 	}