Convert GET to POST requests #140

Author: DaanHoogland

cmd/network.go

@@ -233,13 +233,14 @@ func NewAPIRequest(r *Request, api string, args []string, isAsync bool) (map[str
 	requestURL := fmt.Sprintf("%s?%s", r.Config.ActiveProfile.URL, encodedParams)
 	config.Debug("NewAPIRequest API request URL:", requestURL)
 
-	response, err := r.Client().Get(requestURL)
-	if err != nil {
+	var response *http.Response
+	response,err = executeRequest(r, requestURL, params)
+	if (err != nil) {
 		return nil, err
 	}
 	config.Debug("NewAPIRequest response status code:", response.StatusCode)
 
-	if response != nil && response.StatusCode == http.StatusUnauthorized {
+	if response.StatusCode == http.StatusUnauthorized {
 		r.Client().Jar, _ = cookiejar.New(nil)
 		sessionKey, err := Login(r)
 		if err != nil {
@@ -249,8 +250,9 @@ func NewAPIRequest(r *Request, api string, args []string, isAsync bool) (map[str
 		params.Add("sessionkey", sessionKey)
 		requestURL = fmt.Sprintf("%s?%s", r.Config.ActiveProfile.URL, encodeRequestParams(params))
 		config.Debug("NewAPIRequest API request URL:", requestURL)
-		response, err = r.Client().Get(requestURL)
-		if err != nil {
+
+		response,err = executeRequest(r, requestURL, params)
+		if (err != nil) {
 			return nil, err
 		}
 	}
@@ -277,3 +279,13 @@ func NewAPIRequest(r *Request, api string, args []string, isAsync bool) (map[str
 
 	return nil, errors.New("failed to decode response")
 }
+
+// we can implement further conditions to do POST or GET (or other http commands) here
+func executeRequest(r *Request, requestURL string, params url.Values) (*http.Response, error){
+	if params.Has("password") || params.Has("userdata") {
+		requestURL = fmt.Sprintf("%s", r.Config.ActiveProfile.URL)
+		return r.Client().PostForm(requestURL, params)
+	} else {
+		return r.Client().Get(requestURL)
+	}
+}