-
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
Fix py3 decoding issues in cyberarkpassword.py #59500
Conversation
This commit fixes the following error while using Python 3: ERROR! Unexpected Exception, this is probably a bug: a bytes-like object is required, not 'str'
@Akasurde Are my changes sufficient for a merge? |
@mback2k LGTM. Need additional pair of eyes to take a look. @maxamillion @samdoran @mkrizek Can someone review this ? Thanks in advance. |
@samdoran Thanks for your review, I just pushed a new commit to this PR to address it. |
I think the errors in the latest Shippable check run are unrelated, but instead related to ansible module podman. Could you please retrigger it? See the following error messages: |
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.
LGTM
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.
Looks good. Please create a changelog fragment and we can merge this. See this fragment as an example.
@samdoran Thanks, I just created the requested changelog fragment. |
SUMMARY
This commit fixes the following error while using Python 3:
ERROR! Unexpected Exception, this is probably a bug: a bytes-like object is required, not 'str'
Fixes #52625
Please backport this change to all still supported Ansible versions.
ISSUE TYPE
COMPONENT NAME
cyberarkpassword lookup plugin