-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
ldap_entry bugfix #43620
ldap_entry bugfix #43620
Conversation
@@ -145,9 +145,9 @@ def _load_attrs(self): | |||
attrs[name] = [] | |||
|
|||
if isinstance(value, list): | |||
attrs[name] = value | |||
attrs[name] = [to_bytes(v) for v in value] |
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.
Please could you use map(to_bytes, value)
instead of the for
loop?
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.
Done
Thanks for your PR, @gustavomcarmo. @ansibot please |
@Akasurde could you please review this PR and merge it? |
Closing and re-opening for CI trigger. |
The fix does not resolve the bug, see updated traceback #45417 |
SUMMARY
Fixes the same issue reported in #39569, but for the ldap_entry module.
ISSUE TYPE
COMPONENT NAME
ldap_entry
ANSIBLE VERSION