🧔🏽 Freddy branch

backend/api/utils/jaas_jwt_builder.py

@@ -1,9 +1,10 @@
 import time, uuid
 from authlib.jose import jwt
 
+
 class JaaSJwtBuilder:
     """
-        The JaaSJwtBuilder class helps with the generation of the JaaS JWT.
+    The JaaSJwtBuilder class helps with the generation of the JaaS JWT.
     """
 
     EXP_TIME_DELAY_SEC = 7200
@@ -13,30 +14,32 @@ class JaaSJwtBuilder:
     # Used as a delay for the nbf claim value.
 
     def __init__(self) -> None:
-        self.header = { 'alg' : 'RS256' }
+        self.header = {"alg": "RS256"}
         self.userClaims = {}
         self.featureClaims = {}
         self.payloadClaims = {}
 
     def withDefaults(self):
         """Returns the JaaSJwtBuilder with default valued claims."""
-        return self.withExpTime(int(time.time() + JaaSJwtBuilder.EXP_TIME_DELAY_SEC)) \
-            .withNbfTime(int(time.time() - JaaSJwtBuilder.NBF_TIME_DELAY_SEC)) \
-                .withLiveStreamingEnabled(True) \
-                    .withRecordingEnabled(True) \
-                        .withOutboundCallEnabled(True) \
-                            .withTranscriptionEnabled(True) \
-                                .withModerator(True) \
-                                    .withRoomName('*') \
-                                        .withUserId(str(uuid.uuid4()))
+        return (
+            self.withExpTime(int(time.time() + JaaSJwtBuilder.EXP_TIME_DELAY_SEC))
+            .withNbfTime(int(time.time() - JaaSJwtBuilder.NBF_TIME_DELAY_SEC))
+            .withLiveStreamingEnabled(True)
+            .withRecordingEnabled(True)
+            .withOutboundCallEnabled(True)
+            .withTranscriptionEnabled(True)
+            .withModerator(True)
+            .withRoomName("*")
+            .withUserId(str(uuid.uuid4()))
+        )
 
     def withApiKey(self, apiKey):
         """
         Returns the JaaSJwtBuilder with the kid claim(apiKey) set.
 
         :param apiKey A string as the API Key https://jaas.8x8.vc/#/apikeys
         """
-        self.header['kid'] = apiKey
+        self.header["kid"] = apiKey
         return self
 
     def withUserAvatar(self, avatarUrl):
@@ -45,7 +48,7 @@ def withUserAvatar(self, avatarUrl):
 
         :param avatarUrl A string representing the url to get the user avatar.
         """
-        self.userClaims['avatar'] = avatarUrl
+        self.userClaims["avatar"] = avatarUrl
         return self
 
     def withModerator(self, isModerator):
@@ -54,7 +57,7 @@ def withModerator(self, isModerator):
 
         :param isModerator A boolean if set to True, user is moderator and False otherwise.
         """
-        self.userClaims['moderator'] = 'true' if isModerator == True else 'false'
+        self.userClaims["moderator"] = "true" if isModerator == True else "false"
         return self
 
     def withUserName(self, userName):
@@ -63,7 +66,7 @@ def withUserName(self, userName):
 
         :param userName A string representing the user's name.
         """
-        self.userClaims['name'] = userName
+        self.userClaims["name"] = userName
         return self
 
     def withUserEmail(self, userEmail):
@@ -72,7 +75,7 @@ def withUserEmail(self, userEmail):
 
         :param userEmail A string representing the user's email address.
         """
-        self.userClaims['email'] = userEmail
+        self.userClaims["email"] = userEmail
         return self
 
     def withLiveStreamingEnabled(self, isEnabled):
@@ -81,7 +84,7 @@ def withLiveStreamingEnabled(self, isEnabled):
 
         :param isEnabled A boolean if set to True, live streaming is enabled and False otherwise.
         """
-        self.featureClaims['livestreaming'] = 'true' if isEnabled == True else 'false'
+        self.featureClaims["livestreaming"] = "true" if isEnabled == True else "false"
         return self
 
     def withRecordingEnabled(self, isEnabled):
@@ -90,7 +93,7 @@ def withRecordingEnabled(self, isEnabled):
 
         :param isEnabled A boolean if set to True, recording is enabled and False otherwise.
         """
-        self.featureClaims['recording'] = 'true' if isEnabled == True else 'false'
+        self.featureClaims["recording"] = "true" if isEnabled == True else "false"
         return self
 
     def withTranscriptionEnabled(self, isEnabled):
@@ -99,7 +102,7 @@ def withTranscriptionEnabled(self, isEnabled):
 
         :param isEnabled A boolean if set to True, transcription is enabled and False otherwise.
         """
-        self.featureClaims['transcription'] = 'true' if isEnabled == True else 'false'
+        self.featureClaims["transcription"] = "true" if isEnabled == True else "false"
         return self
 
     def withOutboundCallEnabled(self, isEnabled):
@@ -108,7 +111,7 @@ def withOutboundCallEnabled(self, isEnabled):
 
         :param isEnabled A boolean if set to True, outbound calls are enabled and False otherwise.
         """
-        self.featureClaims['outbound-call'] = 'true' if isEnabled == True else 'false'
+        self.featureClaims["outbound-call"] = "true" if isEnabled == True else "false"
         return self
 
     def withExpTime(self, expTime):
@@ -117,7 +120,7 @@ def withExpTime(self, expTime):
 
         :param expTime Unix time in seconds since epochs plus a delay. Expiration time of the JWT.
         """
-        self.payloadClaims['exp'] = expTime
+        self.payloadClaims["exp"] = expTime
         return self
 
     def withNbfTime(self, nbfTime):
@@ -126,7 +129,7 @@ def withNbfTime(self, nbfTime):
 
         :param nbfTime Unix time in seconds since epochs.
         """
-        self.payloadClaims['nbfTime'] = nbfTime
+        self.payloadClaims["nbfTime"] = nbfTime
         return self
 
     def withRoomName(self, roomName):
@@ -135,7 +138,7 @@ def withRoomName(self, roomName):
 
         :param roomName A string representing the room to join.
         """
-        self.payloadClaims['room'] = roomName
+        self.payloadClaims["room"] = roomName
         return self
 
     def withAppID(self, AppId):
@@ -144,7 +147,7 @@ def withAppID(self, AppId):
 
         :param AppId A string representing the unique AppID (previously tenant).
         """
-        self.payloadClaims['sub'] = AppId
+        self.payloadClaims["sub"] = AppId
         return self
 
     def withUserId(self, userId):
@@ -153,7 +156,7 @@ def withUserId(self, userId):
 
         :param A string representing the user, should be unique from your side.
         """
-        self.userClaims['id'] = userId
+        self.userClaims["id"] = userId
         return self
 
     def signWith(self, key):
@@ -162,8 +165,8 @@ def signWith(self, key):
 
         :param key A string representing the private key in PEM format.
         """
-        context = { 'user': self.userClaims, 'features': self.featureClaims }
-        self.payloadClaims['context'] = context
-        self.payloadClaims['iss'] = 'chat'
-        self.payloadClaims['aud'] = 'jitsi'
+        context = {"user": self.userClaims, "features": self.featureClaims}
+        self.payloadClaims["context"] = context
+        self.payloadClaims["iss"] = "chat"
+        self.payloadClaims["aud"] = "jitsi"
         return jwt.encode(self.header, self.payloadClaims, key)

backend/api/views/meeting.py

@@ -14,7 +14,8 @@
 
 API_KEY = os.environ.get("EIGHT_X_EIGHT_API_KEY")
 APP_ID = os.environ.get("EIGHT_X_EIGHT_APP_ID")
-ENCODED_PRIVATE_KEY = os.environ.get('EIGHT_X_EIGHT_ENCODED_PRIVATE_KEY')
+ENCODED_PRIVATE_KEY = os.environ.get("EIGHT_X_EIGHT_ENCODED_PRIVATE_KEY")
+
 
 @meeting.route("/generateToken", methods=["GET"])
 def generateToken():
@@ -23,17 +24,21 @@ def generateToken():
         PRIVATE_KEY = base64.b64decode(ENCODED_PRIVATE_KEY)
         print(PRIVATE_KEY)
         jaasJwt = JaaSJwtBuilder()
-        token = jaasJwt.withDefaults() \
-            .withApiKey(API_KEY) \
-                .withUserName("User Name") \
-                    .withUserEmail("[email protected]") \
-                        .withModerator(False) \
-                            .withAppID(APP_ID) \
-                                .withUserAvatar("https://asda.com/avatar") \
-                                    .signWith(PRIVATE_KEY)
-
-        return create_response(data={"token": token.decode('utf-8'), "appID": APP_ID})
+        token = (
+            jaasJwt.withDefaults()
+            .withApiKey(API_KEY)
+            .withUserName("User Name")
+            .withUserEmail("[email protected]")
+            .withModerator(False)
+            .withAppID(APP_ID)
+            .withUserAvatar("https://asda.com/avatar")
+            .signWith(PRIVATE_KEY)
+        )
+
+        return create_response(data={"token": token.decode("utf-8"), "appID": APP_ID})
 
     except Exception as error:
         print(error)
-        return create_response(status=422, message=f'Failed to generate token for meeting')
+        return create_response(
+            status=422, message=f"Failed to generate token for meeting"
+        )

backend/tests/test_explore.py

@@ -19,7 +19,7 @@ def test_find_mentee(client):
 
     accounts = result["accounts"]
 
-# There is no need to write assertion on db level data because its change frequently (Database assertion)
+    # There is no need to write assertion on db level data because its change frequently (Database assertion)
     # get the public mentee instances in the database
     mentee_users = MenteeProfile.objects.filter(is_private=False).count()
 
@@ -41,7 +41,7 @@ def test_find_mentor(client):
     assert "accounts" in result
 
     accounts = result["accounts"]
-    #get the mentor instances in the database
+    # get the mentor instances in the database
     mentor_users = MentorProfile.objects.count()
     assert len(accounts) > 0
     assert (