Merge pull request #912 from SoftLayer/issues911

Updated user details objectMask to properly get all required fields

Author: allmightyspiff

.travis.yml

@@ -17,7 +17,7 @@ jobs:
       install:
         # Required to install detect-secrets
         - echo -e "machine github.ibm.com\n  login $CI_USER_TOKEN" > ~/.netrc
-        - curl -sfL https://raw.githubusercontent.com/securego/gosec/master/install.sh | sh -s -- -b $GOPATH/bin
+        - curl -sfL https://raw.githubusercontent.com/securego/gosec/master/install.sh | sh -s -- -d -b $GOPATH/bin
       before_script:
         - go vet $(go list ./... | grep -v "fixtures" | grep -v "vendor")
         - go test $(go list ./... | grep -v "fixtures" | grep -v "vendor")

plugin/commands/user/details.go

@@ -78,7 +78,11 @@ func (cmd *DetailsCommand) Run(args []string) error {
 	logins := cmd.Logins
 	events := cmd.Events
 
-	object_mask := "userStatus[name],parent[id,username],apiAuthenticationKeys[authenticationKey],sslVpnAllowedFlag"
+	object_mask := `mask[
+firstName, lastName, email, openIdConnectUserName, address1, address2, city, state, country, postalCode,
+companyName, createDate, officePhone, id, username, parent, successfulLogins,
+userStatus[name],parent[id,username],apiAuthenticationKeys[authenticationKey],sslVpnAllowedFlag
+]`
 	user, err := cmd.UserManager.GetUser(id, object_mask)
 	userInfo.User = user
 	if err != nil {