From 03f35bd4f547da5843fab755ca678c01800aabed Mon Sep 17 00:00:00 2001
From: Sebastian Thiel <byronimo@gmail.com>
Date: Tue, 21 Apr 2015 12:18:50 +0200
Subject: [PATCH] feat(CLI): --debug-auth flag

* Allow to see all authentication related communication, similar to
  --debug flag otherwise.
* fixed broken generator when handling request value parsing.

Fixes #70
---
 src/mako/cli/lib/docopt.mako |  3 +++
 src/mako/cli/lib/engine.mako | 34 +++++++++++++++++++---------------
 2 files changed, 22 insertions(+), 15 deletions(-)

diff --git a/src/mako/cli/lib/docopt.mako b/src/mako/cli/lib/docopt.mako
index ced012a532..5264a03403 100644
--- a/src/mako/cli/lib/docopt.mako
+++ b/src/mako/cli/lib/docopt.mako
@@ -69,5 +69,8 @@ Configuration:
   --debug
             Output all server communication to standard error. `tx` and `rx` are placed into 
             the same stream.
+  --debug-auth
+            Output all communication related to authentication to standard error. `tx` and `rx` are placed into 
+            the same stream.
 ");
 </%def>
\ No newline at end of file
diff --git a/src/mako/cli/lib/engine.mako b/src/mako/cli/lib/engine.mako
index efc14ff85a..5958dac3b1 100644
--- a/src/mako/cli/lib/engine.mako
+++ b/src/mako/cli/lib/engine.mako
@@ -106,21 +106,15 @@ self.opt.${cmd_ident(method)} {
             }
         };
 
-        let auth = Authenticator::new(&secret, DefaultAuthenticatorDelegate,
-                                      hyper::Client::new(),
-                                      JsonTokenStorage {
-                                        program_name: "${util.program_name()}",
-                                        db_dir: config_dir.clone(),
-                                      }, None);
+        let auth = Authenticator::new(  &secret, DefaultAuthenticatorDelegate,
+                                        ${self._debug_client('debug_auth') | indent_all_but_first_by(10)},
+                                        JsonTokenStorage {
+                                          program_name: "${util.program_name()}",
+                                          db_dir: config_dir.clone(),
+                                        }, None);
 
         let client = 
-            if opt.flag_debug {
-                hyper::Client::with_connector(mock::TeeConnector {
-                        connector: hyper::net::HttpConnector(None) 
-                    })
-            } else {
-                hyper::Client::new()
-            };
+            ${self._debug_client('debug') | indent_all_but_first_by(3)};
         let engine = Engine {
             opt: opt,
             hub: ${hub_type_name}::new(client, auth),
@@ -140,6 +134,16 @@ self.opt.${cmd_ident(method)} {
 }
 </%def>
 
+<%def name="_debug_client(flag_name)" buffered="True">\
+if opt.flag_${flag_name} {
+    hyper::Client::with_connector(mock::TeeConnector {
+            connector: hyper::net::HttpConnector(None) 
+        })
+} else {
+    hyper::Client::new()
+}\
+</%def>
+
 <%def name="_method_call_impl(c, resource, method)" buffered="True">\
 <%
     mc = new_method_context(resource, method, c)
@@ -247,7 +251,7 @@ ${value_unwrap}\
 }
 % endif # handle call parameters
 % if mc.request_value:
-${self._request_value_impl(c, request_cli_schema)}\
+${self._request_value_impl(c, request_cli_schema, request_prop_type)}\
 % endif # handle struct parsing
 % if mc.media_params:
 let protocol = 
@@ -319,7 +323,7 @@ if dry_run {
 }\
 </%def>
 
-<%def name="_request_value_impl(c, request_cli_schema)">
+<%def name="_request_value_impl(c, request_cli_schema, request_prop_type)">
 <%
     allow_optionals_fn = lambda s: is_schema_with_optionals(schema_markers(s, c, transitive=False))