1

opentelekomcloud · Nov 29, 2024 · c9b069f · c9b069f
1 parent 12bde9d
commit c9b069f
Showing 1 changed file with 8 additions and 23 deletions.
diff --git a/openstack/obs/convert.go b/openstack/obs/convert.go
@@ -82,31 +82,16 @@ func convertGrantToXml(grant Grant, isObs bool, isBucket bool) string {
 		xml = append(xml, fmt.Sprintf("<Grant><Grantee xsi:type=\"%s\" xmlns:xsi=\"http://www.w3.org/2001/XMLSchema-instance\">", grant.Grantee.Type))
 	}
 
-	if grant.Grantee.Type == GranteeUser {
-		if isObs {
-			xml = append(xml, "<Grant><Grantee>")
-		}
-		if grant.Grantee.ID != "" {
-			granteeID := XmlTranscoding(grant.Grantee.ID)
-			xml = append(xml, fmt.Sprintf("<ID>%s</ID>", granteeID))
-		}
-		if !isObs && grant.Grantee.DisplayName != "" {
-			granteeDisplayName := XmlTranscoding(grant.Grantee.DisplayName)
-			xml = append(xml, fmt.Sprintf("<DisplayName>%s</DisplayName>", granteeDisplayName))
-		}
+	if !isObs {
+		xml = append(xml, fmt.Sprintf("<Grant><Grantee xsi:type=\"%s\" xmlns:xsi=\"http://www.w3.org/2001/XMLSchema-instance\">", grant.Grantee.Type))
+		xml = append(xml, prepareGrantURI(grant))
+		xml = append(xml, "</Grantee>")
+	} else if grant.Grantee.URI == GroupAllUsers {
+		xml = append(xml, "<Grant><Grantee>")
+		xml = append(xml, "<Canned>Everyone</Canned>")
 		xml = append(xml, "</Grantee>")
 	} else {
-		if !isObs {
-			xml = append(xml, fmt.Sprintf("<Grant><Grantee xsi:type=\"%s\" xmlns:xsi=\"http://www.w3.org/2001/XMLSchema-instance\">", grant.Grantee.Type))
-			xml = append(xml, prepareGrantURI(grant))
-			xml = append(xml, "</Grantee>")
-		} else if grant.Grantee.URI == GroupAllUsers {
-			xml = append(xml, "<Grant><Grantee>")
-			xml = append(xml, "<Canned>Everyone</Canned>")
-			xml = append(xml, "</Grantee>")
-		} else {
-			return strings.Join(xml, "")
-		}
+		return strings.Join(xml, "")
 	}
 
 	xml = append(xml, fmt.Sprintf("<Permission>%s</Permission>", grant.Permission))