From ee7f7a82a8074f45dbfd379c91a182821f8c77d5 Mon Sep 17 00:00:00 2001 From: Greg DeKoenigsberg Date: Fri, 28 Aug 2015 15:19:54 -0400 Subject: [PATCH] Update developing_modules.rst More specific about urllib2 restrictions --- docsite/rst/developing_modules.rst | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/docsite/rst/developing_modules.rst b/docsite/rst/developing_modules.rst index 06af530bce7..7e77f8fc745 100644 --- a/docsite/rst/developing_modules.rst +++ b/docsite/rst/developing_modules.rst @@ -501,9 +501,8 @@ Module checklist serializable. A common pitfall is to try returning an object via exit_json(). Instead, convert the fields you need from the object into the fields of a dictionary and return the dictionary. -* Do not use urllib2 to handle urls. urllib2 does not natively verify TLS - certificates and so is insecure for https. Instead, use either fetch_url or - open_url from ansible.module_utils.urls. +* When fetching URLs, please use either fetch_url or open_url from ansible.module_utils.urls + rather than urllib2; urllib2 does not natively verify TLS certificates and so is insecure for https. Windows modules checklist `````````````````````````