config: fix setting postrequest config

PR apache#161 introduced `postrequest` config but there was no way to update that config other than updating the config file.
This PR fixes the behaviour. Also adds a log to highlight what HTTP
method is used.

Signed-off-by: Abhishek Kumar <[email protected]>

Author: shwstppr

cmd/network.go

@@ -411,9 +411,11 @@ func NewAPIRequest(r *Request, api string, args []string, isAsync bool) (map[str
 func executeRequest(r *Request, requestURL string, params url.Values) (*http.Response, error) {
 	config.SetupContext(r.Config)
 	if params.Has("password") || params.Has("userdata") || r.Config.Core.PostRequest {
-		requestURL = fmt.Sprintf("%s", r.Config.ActiveProfile.URL)
+		requestURL = r.Config.ActiveProfile.URL
+		config.Debug("Using HTTP POST for the request: ", requestURL)
 		return r.Client().PostForm(requestURL, params)
 	}
+	config.Debug("Using HTTP GET for the request: ", requestURL)
 	req, _ := http.NewRequestWithContext(*r.Config.Context, "GET", requestURL, nil)
 	return r.Client().Do(req)
 }

cmd/set.go

@@ -45,6 +45,7 @@ func init() {
 			"verifycert":   {"true", "false"},
 			"debug":        {"true", "false"},
 			"autocomplete": {"true", "false"},
+			"postrequest":  {"true", "false"},
 		},
 		Handle: func(r *Request) error {
 			if len(r.Args) < 1 {

config/config.go

@@ -76,7 +76,7 @@ type Core struct {
 	VerifyCert   bool   `ini:"verifycert"`
 	ProfileName  string `ini:"profile"`
 	AutoComplete bool   `ini:"autocomplete"`
-	PostRequest  bool   `ini:postrequest`
+	PostRequest  bool   `ini:"postrequest"`
 }
 
 // Config describes CLI config file and default options
@@ -401,6 +401,8 @@ func (c *Config) UpdateConfig(key string, value string, update bool) {
 		}
 	case "autocomplete":
 		c.Core.AutoComplete = value == "true"
+	case "postrequest":
+		c.Core.PostRequest = value == "true"
 	default:
 		fmt.Println("Invalid option provided:", key)
 		return