You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by ro...@apache.org on 2019/06/20 09:55:18 UTC

[cloudstack-cloudmonkey] branch master updated: cmd: fix bug around required params check

This is an automated email from the ASF dual-hosted git repository.

rohit pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/cloudstack-cloudmonkey.git


The following commit(s) were added to refs/heads/master by this push:
     new 365e10e  cmd: fix bug around required params check
365e10e is described below

commit 365e10eae61f9135bf1a61aca4ce9087b2870e10
Author: Rohit Yadav <ro...@shapeblue.com>
AuthorDate: Thu Jun 20 15:23:48 2019 +0530

    cmd: fix bug around required params check
    
    This fixes issue for map type of required parameters as seen in the
    createTags API where previously an API request would fail when
    tags[0].<something> was passed.
    
    Signed-off-by: Rohit Yadav <ro...@shapeblue.com>
---
 cmd/api.go | 1 +
 1 file changed, 1 insertion(+)

diff --git a/cmd/api.go b/cmd/api.go
index b84f919..01ed2c3 100644
--- a/cmd/api.go
+++ b/cmd/api.go
@@ -60,6 +60,7 @@ func init() {
 
 			var missingArgs []string
 			for _, required := range api.RequiredArgs {
+				required = strings.ReplaceAll(required, "=", "")
 				provided := false
 				for _, arg := range apiArgs {
 					if strings.Contains(arg, "=") && strings.HasPrefix(arg, required) {