Skip to content

fix(greeter): reconnect to DDM on service changes - #1154

Merged
zccrs merged 1 commit into
linuxdeepin:masterfrom
zorowk:fix_ddm_restart
Jul 16, 2026
Merged

fix(greeter): reconnect to DDM on service changes#1154
zccrs merged 1 commit into
linuxdeepin:masterfrom
zorowk:fix_ddm_restart

Conversation

@zorowk

@zorowk zorowk commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Watch the display manager D-Bus lifecycle. Close the stale authentication socket when the service disappears and refresh AuthInfo using the existing connection path when it returns.

Log: DDM 成功重启后,旧认证 socket 会失效。Treeland 监听
org.deepin.DisplayManager 的注销和注册,在服务注销时关闭旧 socket, 并在服务重新注册后重新获取 AuthInfo()、连接新的认证 socket。
该机制只处理 DDM 成功恢复后的认证链路,不处理 start-limit-hit。

PMS:
Influence: Affects greeter-to-DDM authentication socket recovery after a successful DDM restart.

需要合入ddm相关的修改 linuxdeepin/ddm#104

Watch the display manager D-Bus lifecycle. Close the stale
authentication socket when the service disappears and refresh
AuthInfo using the existing connection path when it returns.

Log: DDM 成功重启后,旧认证 socket 会失效。Treeland 监听
org.deepin.DisplayManager 的注销和注册,在服务注销时关闭旧 socket,
并在服务重新注册后重新获取 AuthInfo()、连接新的认证 socket。
该机制只处理 DDM 成功恢复后的认证链路,不处理 start-limit-hit。

PMS:
Influence: Affects greeter-to-DDM authentication socket recovery after a successful DDM restart.

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry @zorowk, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

Comment thread src/greeter/greeterproxy.cpp
@deepin-ci-robot

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: zccrs, zorowk

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@zccrs
zccrs merged commit 9979373 into linuxdeepin:master Jul 16, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants