Merge pull request #17 from keystone-scim/fix-aad-missing-emails-issue

Handle case with missing emails from AAD
keystone-scim · Aug 16, 2022 · b861160 · b861160
2 parents 38a2ac1 + 6dad725
commit b861160
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/keystone/__init__.py b/keystone/__init__.py
@@ -5,7 +5,7 @@
 _red = "\033[0;31m"
 _nc = "\033[0m"
 
-VERSION = "0.1.0-rc.1"
+VERSION = "0.1.0-rc.2"
 LOGO = """
                             ..............                            
                            .--------------. :                         

diff --git a/keystone/store/postgresql_store.py b/keystone/store/postgresql_store.py
@@ -306,14 +306,15 @@ async def _create_user(self, resource: Dict):
             active=resource.get("active"),
             customAttributes=custom_schemas
         ).returning()
+        emails = resource.get("emails", [{"primary": True, "value": resource.get("userName"), "type": "work"}])
         insert_emails = insert(tbl.user_emails).values([
             {
                 "id": str(uuid.uuid4()),
                 "userId": user_id,
                 "primary": email.get("primary", True),
                 "value": email.get("value"),
                 "type": email.get("type")
-            } for email in resource.get("emails")
+            } for email in emails
         ])
         engine = await self.get_engine()
         async with engine.acquire() as conn: