@@ -35,19 +35,24 @@ func cmdConfigSet(c *cli.Context) (e error) {
3535 if e == nil {
3636 logger .Debug (fmt .Sprintf ("CONFIG SET FINISH: %v" , time .Since (start )))
3737 } else {
38- logger .Error (fmt .Sprintf ("CONFIG SET ERROR: %v" , e . Error () ))
38+ logger .Error (fmt .Sprintf ("CONFIG SET ERROR: %v" , e ))
3939 }
4040 }()
4141 cfg := config .Get (c .Context )
42+
4243 section , key , err := parseConfigPath (c )
4344 if err != nil {
44- return cli .Exit (color .RedString ("Unable to set config value: %s" , err ), 1 )
45+ logger .Error (fmt .Sprintf ("Error parsing config path: %v" , err ))
46+ return cli .Exit (color .RedString ("Unable to set config value: %v" , err ), 1 )
4547 }
48+
4649 value := strings .Join (c .Args ().Tail (), " " )
4750 cfg .SetValue (section , key , value )
4851 if err := cfg .Save (c .Context ); err != nil {
49- return cli .Exit (color .RedString ("Unable to set config value: %s" , err ), 1 )
52+ logger .Error (fmt .Sprintf ("Error saving config: %v" , err ))
53+ return cli .Exit (color .RedString ("Unable to set config value: %v" , err ), 1 )
5054 }
55+
5156 return nil
5257}
5358
@@ -60,19 +65,23 @@ func cmdConfigGet(c *cli.Context) (e error) {
6065 if e == nil {
6166 logger .Debug (fmt .Sprintf ("CONFIG GET FINISH: %v" , time .Since (start )))
6267 } else {
63- logger .Error (fmt .Sprintf ("CONFIG GET ERROR: %v" , e . Error () ))
68+ logger .Error (fmt .Sprintf ("CONFIG GET ERROR: %v" , e ))
6469 }
6570 }()
6671 cfg := config .Get (c .Context )
72+
6773 section , key , err := parseConfigPath (c )
6874 if err != nil {
69- return cli .Exit (color .RedString ("Unable to get config value: %s" , err ), 1 )
75+ logger .Error (fmt .Sprintf ("Error parsing config path: %v" , err ))
76+ return cli .Exit (color .RedString ("Unable to get config value: %v" , err ), 1 )
7077 }
78+
7179 val , _ := cfg .GetValue (section , key )
7280 if _ , err := terminal .Get (c .Context ).Writeln (val ); err != nil {
7381 return err
7482 }
7583 logger .Debug (val )
84+
7685 return nil
7786}
7887
@@ -85,18 +94,20 @@ func cmdConfigUnset(c *cli.Context) (e error) {
8594 if e == nil {
8695 logger .Debug (fmt .Sprintf ("CONFIG UNSET FINISH: %v" , time .Since (start )))
8796 } else {
88- logger .Error (fmt .Sprintf ("CONFIG UNSET ERROR: %v" , e . Error () ))
97+ logger .Error (fmt .Sprintf ("CONFIG UNSET ERROR: %v" , e ))
8998 }
9099 }()
91100 cfg := config .Get (c .Context )
92101 section , key , err := parseConfigPath (c )
93102 if err != nil {
94- return cli .Exit (color .RedString ("Unable to unset config value: %s" , err ), 1 )
103+ logger .Error (fmt .Sprintf ("Error parsing config path: %v" , err ))
104+ return cli .Exit (color .RedString ("Unable to unset config value: %v" , err ), 1 )
95105 }
96106
97107 cfg .UnsetValue (section , key )
98108 if err := cfg .Save (c .Context ); err != nil {
99- return cli .Exit (color .RedString ("Unable to set config value: %s" , err ), 1 )
109+ logger .Error (fmt .Sprintf ("Error saving config: %v" , err ))
110+ return cli .Exit (color .RedString ("Unable to set config value: %v" , err ), 1 )
100111 }
101112 return nil
102113}
@@ -110,7 +121,7 @@ func cmdConfigList(c *cli.Context) (e error) {
110121 if e == nil {
111122 logger .Debug (fmt .Sprintf ("CONFIG LIST FINISH: %v" , time .Since (start )))
112123 } else {
113- logger .Error (fmt .Sprintf ("CONFIG LIST ERROR: %v" , e . Error () ))
124+ logger .Error (fmt .Sprintf ("CONFIG LIST ERROR: %v" , e ))
114125 }
115126 }()
116127 cfg := config .Get (c .Context )
0 commit comments