Skip to content

Conversation

@rafalgajda-silvair
Copy link
Contributor

This patch fixes the way introspection generates node structure for requested path.

Lets export more than one interface under similar path, eg.:
'/test/path'
'/test/path/child'

If a client introspects the root path '/' the response will include duplicated child node:
<node name="/">
<node name="com" />
<node name="com" />
</node>"

Client recursively asking and building interface tree will cause n^2 visible service interfaces where n is a number of actual service interfaces exported.
This patch fixes this behavior (test case included)

@acrisci
Copy link
Member

acrisci commented Oct 3, 2019

👍

@acrisci acrisci merged commit 8990e57 into altdesktop:master Oct 3, 2019
@rafalgajda-silvair rafalgajda-silvair deleted the fix_export_multiple_paths branch October 3, 2019 13:54
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.

2 participants