diff options
author | Ganesh Nalawade <ganesh634@gmail.com> | 2019-04-03 06:10:31 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-04-03 06:10:31 +0200 |
commit | d100faedb36201cb30dda02bcf6ff226710b286b (patch) | |
tree | bdf2e9ae21907e47540f3a41604916f3c1527491 | |
parent | Update azure_rm_managed_disk.py --add zones (#53788) (diff) | |
download | ansible-d100faedb36201cb30dda02bcf6ff226710b286b.tar.xz ansible-d100faedb36201cb30dda02bcf6ff226710b286b.zip |
Add check to log jsonrpc request/response (#54606)
Fixes #54605
Add check in ansible-connection to check if persistent
log message is enabled, if yes log the jsonrpc request
in log file.
With the current approach the _messages queue in `NetworkConnectionBase`
is getting overwritten and response for `pop_message` api inovcation from
ansible-connection is not recevied at the module side.
-rwxr-xr-x | bin/ansible-connection | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/bin/ansible-connection b/bin/ansible-connection index 7f35aa981e..2b4855410e 100755 --- a/bin/ansible-connection +++ b/bin/ansible-connection @@ -136,13 +136,18 @@ class ConnectionProcess(object): data = recv_data(s) if not data: break - self.connection._log_messages("jsonrpc request: %s" % data) + log_messages = self.connection.get_option('persistent_log_messages') + + if log_messages: + display.display("jsonrpc request: %s" % data, log_only=True) signal.alarm(self.connection.get_option('persistent_command_timeout')) resp = self.srv.handle_request(data) signal.alarm(0) - self.connection._log_messages("jsonrpc response: %s" % resp) + if log_messages: + display.display("jsonrpc response: %s" % resp, log_only=True) + send_data(s, to_bytes(resp)) s.close() |