Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DBus interface method overloads do not work well #1722

Open
GooseBravo101 opened this issue Mar 4, 2025 · 0 comments
Open

DBus interface method overloads do not work well #1722

GooseBravo101 opened this issue Mar 4, 2025 · 0 comments

Comments

@GooseBravo101
Copy link

After several rebuilds of signal-cli, I noticed that the client app, written in python, gets confused about overloaded methods of signal-cli, such as sendMessage. Sometimes it says that the sendMessage(message<s>, attachments<as>, recipient<s>) -> timestamp<x> method does not exist, sometimes it's sendMessage(message<s>, attachments<as>, recipients<as>) -> timestamp<x>.

Some rationale on documentation side: D-Bus does not support method overloading, only one method or signal should exist with each name.

I propose splitting such overloaded methods so they have different names

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

No branches or pull requests

1 participant