api module should PUT to server with Content-Length HTTP header

Signed-off-by: Justin Cinkelj <justin.cinkelj@xlab.si>

Author: justinc1

plugins/modules/api.py

@@ -180,6 +180,7 @@
 
 
 from ansible.module_utils.basic import AnsibleModule
+import os
 
 from ..module_utils import errors, arguments
 from ..module_utils.client import Client
@@ -260,6 +261,7 @@ def delete_record(module, rest_client):
 
 
 def put_record(module, rest_client):
+    file_size = os.stat(module.params["source"]).st_size
     with open(module.params["source"], "rb") as source_file:
         result = rest_client.put_record(
             endpoint=module.params["endpoint"],
@@ -271,6 +273,7 @@ def put_record(module, rest_client):
             headers={
                 "Content-Type": "application/octet-stream",
                 "Accept": "application/json",
+                "Content-Length": file_size,
             },
         )
     return True, result

tests/unit/plugins/modules/test_api.py

@@ -97,7 +97,9 @@ def test_put_method(self, create_module, rest_client, mocker):
                 data=dict(),
             )
         )
-        mocker.patch("builtins.open", mocker.mock_open(read_data="this-data"))
+        this_data = "this-data"
+        mocker.patch("builtins.open", mocker.mock_open(read_data=this_data))
+        mocker.patch("os.stat").return_value = mocker.Mock(st_size=len(this_data))
         rest_client.put_record.return_value = "this-value"
         result = api.put_record(module, rest_client)
         assert result == (True, "this-value")