Skip to content
New issue

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

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

Already on GitHub? # to your account

Persistent search result entry does not contain controls #69

Closed
pavelhoral opened this issue Feb 4, 2023 · 1 comment
Closed

Persistent search result entry does not contain controls #69

pavelhoral opened this issue Feb 4, 2023 · 1 comment
Labels

Comments

@pavelhoral
Copy link
Member

When doing persistent search via LDAPConnectionHandler2 the result entry does not contain EntryChangeNotification control. This causes issues in Wren:AM that uses that information in SMS component.

I have already debugged what is happening and the issue is that LDAPClientConnection2 is mapping search result entry to response using method that leads to calling incorrect overloaded constructor.

Quite obvious fix is to add overloaded method to Responses class. However I would like to verify that it is the correct way and also possibly investigate if there is a place where we can add unit test to test this behaviour.

@pavelhoral pavelhoral self-assigned this Feb 4, 2023
pavelhoral added a commit to orchitech/wrends that referenced this issue Feb 8, 2023
fyrbach added a commit that referenced this issue Feb 8, 2023
Fix persistent search result missing controls. #69
@pavelhoral pavelhoral removed their assignment Feb 8, 2023
@pavelhoral
Copy link
Member Author

The underlying issue has been fixed in #75. Unfortunately without test case... we will add persistent search test to our system test project later.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant