# Buggy code
def url_get(
    module,
    url,
    dest,
    use_proxy,
    last_mod_time,
    force,
    timeout=10,
    headers=None,
    tmp_dest="",
):
    """
    Download data from the url and store in a temporary file.

    Return (tempfile, info about the request)
    """
    if module.check_mode:
        method = "HEAD"
    else:
        method = "GET"

    rsp, info = fetch_url(
        module,
        url,
        use_proxy=use_proxy,
        force=force,
        last_mod_time=last_mod_time,
        timeout=timeout,
        headers=headers,
        method=method,
    )

    if info["status"] == 304:
        module.exit_json(url=url, dest=dest, changed=False, msg=info.get("msg", ""))

    # Exceptions in fetch_url may result in a status -1, the ensures a proper error to the user in all cases
    if info["status"] == -1:
        module.fail_json(msg=info["msg"], url=url, dest=dest)

    if (
        info["status"] != 200
        and not url.startswith("file:/")
        and not (url.startswith("ftp:/") and info.get("msg", "").startswith("OK"))
    ):
        module.fail_json(
            msg="Request failed",
            status_code=info["status"],
            response=info["msg"],
            url=url,
            dest=dest,
        )

    # create a temporary file and copy content to do checksum-based replacement
    if tmp_dest:
        # tmp_dest should be an existing dir
        tmp_dest_is_dir = os.path.isdir(tmp_dest)
        if not tmp_dest_is_dir:
            if os.path.exists(tmp_dest):
                module.fail_json(
                    msg="%s is a file but should be a directory." % tmp_dest
                )
            else:
                module.fail_json(msg="%s directory does not exist." % tmp_dest)

        fd, tempname = tempfile.mkstemp(dir=tmp_dest)
    else:
        fd, tempname = tempfile.mkstemp()

    f = os.fdopen(fd, "wb")
    try:
        shutil.copyfileobj(rsp, f)
    except Exception as e:
        os.remove(tempname)
        module.fail_json(
            msg="failed to create temporary content file: %s" % to_native(e),
            exception=traceback.format_exc(),
        )
    f.close()
    rsp.close()
    return tempname, info


# Traceback (OSError)
ansible-playbook 2.6.0rc1
config file = None
configured module search path = [u'/home/centos/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
ansible python module location = /usr/lib/python2.7/site-packages/ansible
executable location = /usr/bin/ansible-playbook
python version = 2.7.5 (default, Apr 11 2018, 07:36:10) [GCC 4.8.5 20150623 (Red Hat 4.8.5-28)]
No config file found; using defaults
Parsed /home/centos/inventory.ini inventory source with ini plugin

PLAYBOOK: playbook.yml *************************************************************************************************************************************
1 plays in playbook.yml

PLAY [all] *************************************************************************************************************************************************

TASK [Gathering Facts] *************************************************************************************************************************************
task path: /home/centos/playbook.yml:2
<target> ESTABLISH LOCAL CONNECTION FOR USER: centos
<target> EXEC /bin/sh -c 'echo ~centos &amp;&amp; sleep 0'
<target> EXEC /bin/sh -c '( umask 77 &amp;&amp; mkdir -p "` echo /home/centos/.ansible/tmp/ansible-tmp-1528489241.67-154074287834207 `" &amp;&amp; echo ansible-tmp-1528489241.67-154074287834207="` echo /home/centos/.ansible/tmp/ansible-tmp-1528489241.67-154074287834207 `" ) &amp;&amp; sleep 0'
Using module file /usr/lib/python2.7/site-packages/ansible/modules/system/setup.py
<target> PUT /home/centos/.ansible/tmp/ansible-local-26937OUx1qh/tmpsaSW45 TO /home/centos/.ansible/tmp/ansible-tmp-1528489241.67-154074287834207/setup.py
<target> EXEC /bin/sh -c 'chmod u+x /home/centos/.ansible/tmp/ansible-tmp-1528489241.67-154074287834207/ /home/centos/.ansible/tmp/ansible-tmp-1528489241.67-154074287834207/setup.py &amp;&amp; sleep 0'
<target> EXEC /bin/sh -c '/usr/bin/python /home/centos/.ansible/tmp/ansible-tmp-1528489241.67-154074287834207/setup.py &amp;&amp; sleep 0'
<target> EXEC /bin/sh -c 'rm -f -r /home/centos/.ansible/tmp/ansible-tmp-1528489241.67-154074287834207/ > /dev/null 2>&amp;1 &amp;&amp; sleep 0'
ok: [target]
META: ran handlers

TASK [blockinfile] *****************************************************************************************************************************************
task path: /home/centos/playbook.yml:4
<target> ESTABLISH LOCAL CONNECTION FOR USER: centos
<target> EXEC /bin/sh -c 'echo ~centos &amp;&amp; sleep 0'
<target> EXEC /bin/sh -c '( umask 77 &amp;&amp; mkdir -p "` echo /var/tmp/ansible-tmp-1528489242.66-190869334332659 `" &amp;&amp; echo ansible-tmp-1528489242.66-190869334332659="` echo /var/tmp/ansible-tmp-1528489242.66-190869334332659 `" ) &amp;&amp; sleep 0'
Using module file /usr/lib/python2.7/site-packages/ansible/modules/files/blockinfile.py
<target> PUT /home/centos/.ansible/tmp/ansible-local-26937OUx1qh/tmp4y0GYj TO /var/tmp/ansible-tmp-1528489242.66-190869334332659/blockinfile.py
<target> EXEC /bin/sh -c 'setfacl -m u:testuser:r-x /var/tmp/ansible-tmp-1528489242.66-190869334332659/ /var/tmp/ansible-tmp-1528489242.66-190869334332659/blockinfile.py &amp;&amp; sleep 0'
<target> EXEC /bin/sh -c 'sudo -H -S -n -u testuser /bin/sh -c '"'"'echo BECOME-SUCCESS-dkqkzvhlmkxabaonclqypkdsoabveqpw; /usr/bin/python /var/tmp/ansible-tmp-1528489242.66-190869334332659/blockinfile.py'"'"' &amp;&amp; sleep 0'
<target> EXEC /bin/sh -c 'rm -f -r /var/tmp/ansible-tmp-1528489242.66-190869334332659/ > /dev/null 2>&amp;1 &amp;&amp; sleep 0'
The full traceback is:
Traceback (most recent call last):
File "/tmp/ansible_c67IEZ/ansible_module_blockinfile.py", line 351, in <module>
main()
File "/tmp/ansible_c67IEZ/ansible_module_blockinfile.py", line 335, in main
write_changes(module, result, real_path)
File "/tmp/ansible_c67IEZ/ansible_module_blockinfile.py", line 163, in write_changes
tmpfd, tmpfile = tempfile.mkstemp(dir=getattr(module, 'tmpdir', None))
File "/usr/lib64/python2.7/tempfile.py", line 304, in mkstemp
return _mkstemp_inner(dir, prefix, suffix, flags)
File "/usr/lib64/python2.7/tempfile.py", line 239, in _mkstemp_inner
fd = _os.open(file, flags, 0600)
OSError: [Errno 13] Permission denied: '/var/tmp/ansible-tmp-1528489242.66-190869334332659/tmpXemXXr'

fatal: [target]: FAILED! => {
"changed": false,
"module_stderr": "Traceback (most recent call last):\n  File \"/tmp/ansible_c67IEZ/ansible_module_blockinfile.py\", line 351, in <module>\n    main()\n  File \"/tmp/ansible_c67IEZ/ansible_module_blockinfile.py\", line 335, in main\n    write_changes(module, result, real_path)\n  File \"/tmp/ansible_c67IEZ/ansible_module_blockinfile.py\", line 163, in write_changes\n    tmpfd, tmpfile = tempfile.mkstemp(dir=getattr(module, 'tmpdir', None))\n  File \"/usr/lib64/python2.7/tempfile.py\", line 304, in mkstemp\n    return _mkstemp_inner(dir, prefix, suffix, flags)\n  File \"/usr/lib64/python2.7/tempfile.py\", line 239, in _mkstemp_inner\n    fd = _os.open(file, flags, 0600)\nOSError: [Errno 13] Permission denied: '/var/tmp/ansible-tmp-1528489242.66-190869334332659/tmpXemXXr'\n",
"module_stdout": "",
"msg": "MODULE FAILURE",
"rc": 1
}
to retry, use: --limit @/home/centos/playbook.retry

PLAY RECAP *************************************************************************************************************************************************
target                     : ok=1    changed=0    unreachable=0    failed=1

# Fixed code
def url_get(
    module,
    url,
    dest,
    use_proxy,
    last_mod_time,
    force,
    timeout=10,
    headers=None,
    tmp_dest="",
):
    """
    Download data from the url and store in a temporary file.

    Return (tempfile, info about the request)
    """
    if module.check_mode:
        method = "HEAD"
    else:
        method = "GET"

    rsp, info = fetch_url(
        module,
        url,
        use_proxy=use_proxy,
        force=force,
        last_mod_time=last_mod_time,
        timeout=timeout,
        headers=headers,
        method=method,
    )

    if info["status"] == 304:
        module.exit_json(url=url, dest=dest, changed=False, msg=info.get("msg", ""))

    # Exceptions in fetch_url may result in a status -1, the ensures a proper error to the user in all cases
    if info["status"] == -1:
        module.fail_json(msg=info["msg"], url=url, dest=dest)

    if (
        info["status"] != 200
        and not url.startswith("file:/")
        and not (url.startswith("ftp:/") and info.get("msg", "").startswith("OK"))
    ):
        module.fail_json(
            msg="Request failed",
            status_code=info["status"],
            response=info["msg"],
            url=url,
            dest=dest,
        )

    # create a temporary file and copy content to do checksum-based replacement
    if tmp_dest:
        # tmp_dest should be an existing dir
        tmp_dest_is_dir = os.path.isdir(tmp_dest)
        if not tmp_dest_is_dir:
            if os.path.exists(tmp_dest):
                module.fail_json(
                    msg="%s is a file but should be a directory." % tmp_dest
                )
            else:
                module.fail_json(msg="%s directory does not exist." % tmp_dest)
    else:
        tmp_dest = module.tmpdir

    fd, tempname = tempfile.mkstemp(dir=tmp_dest)

    f = os.fdopen(fd, "wb")
    try:
        shutil.copyfileobj(rsp, f)
    except Exception as e:
        os.remove(tempname)
        module.fail_json(
            msg="failed to create temporary content file: %s" % to_native(e),
            exception=traceback.format_exc(),
        )
    f.close()
    rsp.close()
    return tempname, info