Better room naming : UUID + str convert method in utils

tracim/tracim/controllers/jitsi_meet.py

@@ -6,7 +6,7 @@ from tg.predicates import not_anonymous
 from tracim.lib.predicates import current_user_is_reader
 from sqlalchemy.orm.exc import NoResultFound
 
-from unidecode import unidecode
+from tracim.lib.utils import str_as_alpha_num_str
 from tracim.lib.jitsi_meet.jitsi_meet import JitsiMeetRoom
 from tracim.lib.jitsi_meet.jitsi_meet import JitsiTokenConfig
 from tracim.config.app_cfg import CFG
@@ -41,12 +41,15 @@ class JitsiMeetController(TIMRestController):
         dictified_workspace = Context(CTX.WORKSPACE).toDict(workspace,
                                                             'workspace')
 
-        label = unidecode(workspace.label)
-        parsed_label = ''.join(e for e in label if e.isalnum())
         # TODO - G.M - 18-01-2017 -
         # allow to set specific room name from workspace object ?
-        room = "{id}{label}".format(id=workspace.workspace_id,
-                                    label=parsed_label)
+        room = "{uuid}{workspace_id}{workspace_label}".format(
+            uuid=cfg.TRACIM_INSTANCE_UUID,
+            workspace_id=workspace.workspace_id,
+            workspace_label=workspace.label)
+
+        # Jitsi-Meet doesn't like specials_characters
+        room = str_as_alpha_num_str(room)
 
         token = None
         if cfg.JITSI_MEET_USE_TOKEN:

tracim/tracim/lib/utils.py

@@ -16,6 +16,7 @@ from tg.util import LazyString as BaseLazyString
 from tg.util import lazify
 from redis import Redis
 from rq import Queue
+from unidecode import unidecode
 
 from tracim.lib.base import logger
 from webob import Response
@@ -140,6 +141,18 @@ def str_as_bool(string: str) -> bool:
     return bool(string)
 
 
+def str_as_alpha_num_str(unicode_string: str) -> str:
+    """
+    convert unicode string to alpha_num-only string.
+    convert also accented character to ascii equivalent.
+    :param unicode_string:
+    :return:
+    """
+    ascii_string = unidecode(unicode_string)
+    alpha_num_string = ''.join(e for e in ascii_string if e.isalnum())
+    return alpha_num_string
+
+
 class LazyString(BaseLazyString):
     pass