opendev/storyboard
Code Issues Proposed changes

Duplicating Users at Migration Time

Browse Source 
The old login.launchpad.net service has been deprecated
and replaced by/merged with the login.ubuntu.com service.
All out existing users with old OpenID URLs were manually
updated to the new provider hostname but, in at least some
cases, the Launchpad API returns the old URLs for queried
accounts. In an effort to avoid future duplication of
StoryBoard accounts on import, edit the link to match the
ubuntu login link early enough that we still check that it
doesn't already exist before adding it.

Change-Id: Ib3202522d948b2c12cc85dfc12ed5f7f2daf31a9
This commit is contained in:
Kendall Nelson 2018-03-20 15:49:12 -07:00 committed by Jeremy Stanley
parent c67d349dc9
commit ac459828ae
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
storyboard/migrate/launchpad/writer.py
View File

@ -116,6 +116,8 @@ class LaunchpadWriter(object):
display_name = lp_user.display_name display_name = lp_user.display_name
user_link = lp_user.web_link user_link = lp_user.web_link
user_link.replace('login.launchpad.net', 'login.ubuntu.com')
# Resolve the openid. # Resolve the openid.
if user_link not in self._openid_map: if user_link not in self._openid_map:
try: try: