cqrenet/matrix-synapse-rest-password-provider
3
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Update set_profile_displayname to use UserID type

Browse Source 
This is required as the function was updated to take this explicit type,
rather than just the localpart, in matrix-org/synapse#15458 as part of
version 1.83.

This is stacked atop #10 to ensure everything is updated.
This commit is contained in:
Joshua M. Boniface
2023-05-24 23:49:23 -04:00
parent b6bdebbc4a
commit 1010ef4e8f
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
rest_auth_provider.py
View File

@@ -26,6 +26,7 @@ import requests
import time import time
import synapse import synapse
from synapse import module_api from synapse import module_api
from synapse.types import UserID
logger = logging.getLogger(__name__) logger = logging.getLogger(__name__)
@@ -91,7 +92,9 @@ class RestAuthProvider(object):
logger.info("User not authenticated") logger.info("User not authenticated")
return False return False
types_user_id = UserID.from_string(user_id)
localpart = user_id.split(":", 1)[0][1:] localpart = user_id.split(":", 1)[0][1:]
domain = user_id.split(":", 1)[1][1:]
logger.info("User %s authenticated", user_id) logger.info("User %s authenticated", user_id)
registration = False registration = False
@@ -117,7 +120,7 @@ class RestAuthProvider(object):
if "display_name" in profile and ((registration and self.config.setNameOnRegister) or (self.config.setNameOnLogin)): if "display_name" in profile and ((registration and self.config.setNameOnRegister) or (self.config.setNameOnLogin)):
display_name = profile["display_name"] display_name = profile["display_name"]
logger.info("Setting display name to '%s' based on profile data", display_name) logger.info("Setting display name to '%s' based on profile data", display_name)
await store.set_profile_displayname(localpart, display_name) await store.set_profile_displayname(types_user_id, display_name)
else: else:
logger.info("Display name was not set because it was not given or policy restricted it") logger.info("Display name was not set because it was not given or policy restricted it")