Skip to content

fix: release the agent object path on unregister - #530

Draft
cachebag wants to merge 2 commits into
masterfrom
fix/unregister-releases-agent-object-path
Draft

cachebag wants to merge 2 commits into
masterfrom
fix/unregister-releases-agent-object-path

Conversation

@cachebag

@cachebag cachebag commented Aug 20, 2026

Copy link
Copy Markdown
Member

Remove the SecretAgent interface from the object server unconditionally, so the object path is freed even when the NetworkManager Unregister call fails or the daemon is unreachable. The D-Bus error is still propagated after the removal.

@cachebag
cachebag deployed to self-hosted-pr-integration August 20, 2026 14:34 — with GitHub Actions Active
@cachebag
cachebag deployed to self-hosted-pr-integration August 20, 2026 14:39 — with GitHub Actions Active
@cachebag cachebag self-assigned this Aug 20, 2026
@cachebag

Copy link
Copy Markdown
Member Author

Tracking: z-galaxy/zbus#1917

Remove the SecretAgent interface from the object server unconditionally,
so the object path is freed even when the NetworkManager Unregister call
fails or the daemon is unreachable. The D-Bus error is still propagated
after the removal.
@cachebag
cachebag force-pushed the fix/unregister-releases-agent-object-path branch from 2bcf512 to bb75c64 Compare September 2, 2026 14:51
@cachebag
cachebag had a problem deploying to self-hosted-pr-integration September 2, 2026 14:51 — with GitHub Actions Failure
@cachebag
cachebag had a problem deploying to self-hosted-pr-integration September 5, 2026 02:17 — with GitHub Actions Failure
@cachebag
cachebag deployed to self-hosted-pr-integration September 5, 2026 02:28 — with GitHub Actions Active
@freedesktop-rs freedesktop-rs deleted a comment from greptile-apps Bot Sep 17, 2026
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.

1 participant