(Merge 3.4) Issue #23881: urllib.request.ftpwrapper constructor now closes the

socket if the FTP connection failed to fix a ResourceWarning.

diff --cc Lib/urllib/request.py
Simple merge

diff --cc Misc/NEWS
index 7b64a89682e781089ed3711ca26cded6139a45a7,645e1ade1bb1c810b13da88c1f1491e61625fde8..50932081fc09f4a5e1d40165231f0b36b2bbe257
--- 1/Misc/NEWS
--- 2/Misc/NEWS
+++ b/Misc/NEWS
@@@ -19,14 -24,9 +19,17 @@@ Core and Builtin
  Library
  -------
  
+ - Issue #23881: urllib.request.ftpwrapper constructor now closes the socket if
+   the FTP connection failed to fix a ResourceWarning.
+ 
 +- Issue #23853: :meth:`socket.socket.sendall` does no more reset the socket
 +  timeout each time data is sent successfuly. The socket timeout is now the
 +  maximum total duration to send all data.
 +
 +- Issue #22721: An order of multiline pprint output of set or dict containing
 +  orderable and non-orderable elements no longer depends on iteration order of
 +  set or dict.
 +
  - Issue #15133: _tkinter.tkapp.getboolean() now supports Tcl_Obj and always
    returns bool.  tkinter.BooleanVar now validates input values (accepted bool,
    int, str, and Tcl_Obj).  tkinter.BooleanVar.get() now always returns bool.