From 96be75e1774e7309a22337c015985f8d16e21a6e Mon Sep 17 00:00:00 2001
From: Alexandre Aubin <alex.aubin@mailoo.org>
Date: Sun, 15 Mar 2020 22:33:51 +0100
Subject: [PATCH] [fix] Make sure the group permission update contains unique
 elements

---
 src/yunohost/permission.py | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/yunohost/permission.py b/src/yunohost/permission.py
index 9d3d8feda..775a8cd71 100644
--- a/src/yunohost/permission.py
+++ b/src/yunohost/permission.py
@@ -452,6 +452,9 @@ def _update_ldap_group_permission(permission, allowed, sync_perm=True):
         return existing_permission
 
     allowed = [allowed] if not isinstance(allowed, list) else allowed
+    
+    # Guarantee uniqueness of values in allowed, which would otherwise make ldap.update angry.
+    allowed = set(allowed)
 
     try:
         ldap.update('cn=%s,ou=permission' % permission,