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
Fix attribute deleted from LDAP is not immediately reflected even if it is "Always Read Value From LDAP" #15929
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the PR @wadahiro! The fix looks correct to me.
However, testing can be improved a little bit in our new model tests. I created an enhanced version of your test here:mhajas@0d4bac2
Could you please remove your test and add the one from the commit above to this PR?
13f1d34
to
591595b
Compare
591595b
to
5852109
Compare
@mhajas Thank you for your review. I fixed my commit and rebased on the latest main branch. Could you check it again? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @wadahiro, LGTM
@wadahiro The PR now has some conflict, could you please rebase? |
…it is "Always Read Value From LDAP"
@mhajas I rebased it again. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @wadahiro
Fixes #15927