use to_text instead of str in async_status (ansible#45990)

* use to_text instead of str in async_status * used to_native instead as per feedback
lookfirst · Oct 12, 2018 · 3453dac · 3453dac
1 parent e9a7a74
commit 3453dac
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/changelogs/fragments/async_statys_pyx_compat_fix.yml b/changelogs/fragments/async_statys_pyx_compat_fix.yml
@@ -0,0 +1,2 @@
+bugfixes:
+    - use to_text (py2/3 safe) instead of str for 'textualizing' intput in async_status
diff --git a/lib/ansible/modules/utilities/logic/async_status.py b/lib/ansible/modules/utilities/logic/async_status.py
@@ -42,6 +42,7 @@
 import json
 import os
 
+from ansible.module_utils._text import to_native
 from ansible.module_utils.basic import AnsibleModule
 from ansible.module_utils.six import iteritems
 
@@ -93,7 +94,7 @@ def main():
         data['finished'] = 0
 
     # Fix error: TypeError: exit_json() keywords must be strings
-    data = dict([(str(k), v) for k, v in iteritems(data)])
+    data = dict([(to_native(k), v) for k, v in iteritems(data)])
 
     module.exit_json(**data)
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		bugfixes:
		- use to_text (py2/3 safe) instead of str for 'textualizing' intput in async_status