fix: sync Huihui app user homepage profile
This commit was merged in pull request #1.
This commit is contained in:
@@ -238,6 +238,15 @@ class NewsPlatformService:
|
||||
desired_real_name and desired_real_name != sync_real_name,
|
||||
desired_avatar and desired_avatar != sync_avatar,
|
||||
])
|
||||
# usercenter 与 App 的“TA 的主页”不是同一数据源。即使
|
||||
# usercenter 已一致,也必须检查 huihuiuserextend 的主页资料。
|
||||
if not needs_profile_sync:
|
||||
home_ok, home_data = await self.get_huihui_user_home(db, user)
|
||||
needs_profile_sync = (
|
||||
not home_ok
|
||||
or (desired_nickname and home_data.get("name") != desired_nickname)
|
||||
or (desired_avatar and home_data.get("avatar") != desired_avatar)
|
||||
)
|
||||
if needs_profile_sync:
|
||||
ok, err = await self.update_user_profile(
|
||||
db,
|
||||
@@ -1272,10 +1281,11 @@ class NewsPlatformService:
|
||||
if description is not None: body["description"] = description
|
||||
if email is not None: body["email"] = email
|
||||
|
||||
# 会会把“当前用户资料”和“TA 的主页”分开保存:
|
||||
# 会会实际有三份用户资料:
|
||||
# 1. /v2/users/current 更新当前用户资料;
|
||||
# 2. /users/page/{id} 更新公开主页的昵称和头像。
|
||||
# 两处都成功后才允许把本地同步标记为成功。
|
||||
# 2. /users/page/{id} 更新 usercenter 公开资料;
|
||||
# 3. /huihuiuserextend/user 更新 App“TA 的主页”。
|
||||
# 三处都成功且 App 主页回读一致后才允许标记为成功。
|
||||
headers = dict(self._bearer(token))
|
||||
headers["Content-Type"] = "application/json"
|
||||
|
||||
@@ -1302,7 +1312,17 @@ class NewsPlatformService:
|
||||
logger.warning(f"[同步TA的主页] {user.account} 失败: {page_err}")
|
||||
return False, f"TA的主页同步失败: {page_err}"
|
||||
|
||||
# 两个会会接口均成功后再同步本地数据库。
|
||||
home_ok, home_err = await self.update_huihui_user_home(
|
||||
db,
|
||||
user,
|
||||
name=nick_name,
|
||||
avatar=avatar,
|
||||
)
|
||||
if not home_ok:
|
||||
logger.warning(f"[同步App用户主页] {user.account} 失败: {home_err}")
|
||||
return False, f"App用户主页同步失败: {home_err}"
|
||||
|
||||
# 三套会会资料均成功并通过 App 主页回读后再同步本地数据库。
|
||||
local_vals = {}
|
||||
if nick_name is not None: local_vals["nickname"] = nick_name
|
||||
if real_name is not None: local_vals["real_name"] = real_name
|
||||
@@ -1312,7 +1332,7 @@ class NewsPlatformService:
|
||||
await db.execute(update(VirtualUser).where(
|
||||
VirtualUser.id == user.id).values(**local_vals))
|
||||
await db.commit()
|
||||
logger.info(f"✅ 用户 {user.account} 当前资料与TA的主页均已同步")
|
||||
logger.info(f"✅ 用户 {user.account} 三套资料与App用户主页均已同步")
|
||||
return True, ""
|
||||
except Exception as e:
|
||||
logger.warning(f"[修改用户信息] {user.account} 异常: {e}")
|
||||
@@ -1381,6 +1401,89 @@ class NewsPlatformService:
|
||||
except Exception as e:
|
||||
return False, str(e)
|
||||
|
||||
async def get_huihui_user_home(
|
||||
self, db: AsyncSession, user: VirtualUser,
|
||||
) -> tuple[bool, dict | str]:
|
||||
"""回读会会 App `/otherIndex` 实际使用的“TA 的主页”资料。"""
|
||||
sess = await get_session(user.id)
|
||||
if not sess:
|
||||
return False, "用户未登录"
|
||||
platform_uid = sess.get("platform_uid") or user.platform_uid or ""
|
||||
if not platform_uid:
|
||||
return False, "缺少平台用户ID"
|
||||
|
||||
cfg = await self._client(db)
|
||||
api_root = self._api_root(await self._biz_url(db))
|
||||
extra = {"userId": platform_uid}
|
||||
org_id = sess.get("org_id") or cfg.get("orgId") or ""
|
||||
if org_id:
|
||||
extra["orgId"] = org_id
|
||||
params = self._build_form(extra, cfg)
|
||||
try:
|
||||
async with httpx.AsyncClient(timeout=15) as c:
|
||||
r = await c.get(
|
||||
f"{api_root}/huihuiuserextend/user/home/{platform_uid}",
|
||||
params=params,
|
||||
headers=self._bearer(sess.get("token", "")),
|
||||
)
|
||||
d = r.json()
|
||||
if r.status_code == 200 and d.get("code") in [0, 200] and isinstance(d.get("data"), dict):
|
||||
return True, d["data"]
|
||||
return False, d.get("message") or f"HTTP={r.status_code}, code={d.get('code')}"
|
||||
except Exception as e:
|
||||
return False, str(e)
|
||||
|
||||
async def update_huihui_user_home(
|
||||
self, db: AsyncSession, user: VirtualUser,
|
||||
name: str = None, avatar: str = None,
|
||||
) -> tuple[bool, str]:
|
||||
"""写入并回读验证会会 App 真正使用的用户扩展主页资料。"""
|
||||
sess = await get_session(user.id)
|
||||
if not sess:
|
||||
return False, "用户未登录,请先登录"
|
||||
platform_uid = sess.get("platform_uid") or user.platform_uid or ""
|
||||
if not platform_uid:
|
||||
return False, "缺少平台用户ID,请重新登录"
|
||||
|
||||
current_ok, current = await self.get_huihui_user_home(db, user)
|
||||
if not current_ok:
|
||||
return False, f"主页资料回读失败: {current}"
|
||||
extend_id = current.get("id")
|
||||
if not extend_id:
|
||||
return False, "会会用户扩展资料缺少记录ID"
|
||||
|
||||
desired_name = name if name is not None else (user.nickname or "")
|
||||
desired_avatar = avatar if avatar is not None else (user.avatar_url or "")
|
||||
body = {
|
||||
"id": extend_id,
|
||||
"userId": platform_uid,
|
||||
"name": desired_name,
|
||||
"avatar": desired_avatar,
|
||||
}
|
||||
cfg = await self._client(db)
|
||||
api_root = self._api_root(await self._biz_url(db))
|
||||
params = self._build_form({"userId": platform_uid}, cfg)
|
||||
try:
|
||||
async with httpx.AsyncClient(timeout=15) as c:
|
||||
r = await c.patch(
|
||||
f"{api_root}/huihuiuserextend/user",
|
||||
params=params,
|
||||
json=body,
|
||||
headers={**self._bearer(sess.get("token", "")), "Content-Type": "application/json"},
|
||||
)
|
||||
d = r.json()
|
||||
if r.status_code != 200 or d.get("code") not in [0, 200]:
|
||||
return False, d.get("message") or f"HTTP={r.status_code}, code={d.get('code')}"
|
||||
|
||||
verify_ok, verified = await self.get_huihui_user_home(db, user)
|
||||
if not verify_ok:
|
||||
return False, f"写入后回读失败: {verified}"
|
||||
if verified.get("name") != desired_name or verified.get("avatar") != desired_avatar:
|
||||
return False, "写入后App用户主页昵称或头像不一致"
|
||||
return True, ""
|
||||
except Exception as e:
|
||||
return False, str(e)
|
||||
|
||||
async def upload_avatar(
|
||||
self, db: AsyncSession, user: VirtualUser, file_bytes: bytes, filename: str
|
||||
) -> tuple[bool, str]:
|
||||
|
||||
Reference in New Issue
Block a user