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

Make client info available #144

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

tom-cook-veea
Copy link

The main change here is to make the dbus_next.message.Message object available through the context-local current_message proxy. In client code, the typical usage is:

from dbus_next.message_bus import current_message

@method()
def echo_sender() -> 's':
    return current_message.sender

A unit test demonstrating this usage is also included.

This also changes the @method() decorator so that the method can still be called in a non-dbus context and correctly returns its value.

This also fixes a defect in the unit tests; on my Ubuntu 22.04 system, the default session bus socket is not in the abstract socket namespace. This caused test_tcp_connection_with_forwarding to fail as it assumed that the session bus used a socket in the abstract namespace. I have modified it to make it work with either an abstract namespace socket or a filesystem socket.

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