Fix a bug where an unbounded number of credentials wrappers got added.

datalab/utils/_http.py

@@ -19,6 +19,7 @@
 from past.builtins import basestring
 from builtins import object
 
+import copy
 import datetime
 import json
 import urllib.request, urllib.parse, urllib.error
@@ -119,9 +120,12 @@ def request(url, args=None, data=None, headers=None, method=None,
     if method is None:
       method = 'GET'
 
-    # Authorize with credentials if given.
     http = Http.http
+
+    # Authorize with credentials if given.
     if credentials is not None:
+      # Make a copy of the shared http instance before we modify it.
+      http = copy.copy(http)
       http = credentials.authorize(http)
     if stats is not None:
       stats['duration'] = datetime.datetime.utcnow()

google/datalab/utils/_http.py

@@ -19,6 +19,7 @@
 from past.builtins import basestring
 from builtins import object
 
+import copy
 import datetime
 import json
 import urllib.request, urllib.parse, urllib.error
@@ -119,9 +120,12 @@ def request(url, args=None, data=None, headers=None, method=None,
     if method is None:
       method = 'GET'
 
-    # Authorize with credentials if given
     http = Http.http
+
+    # Authorize with credentials if given
     if credentials is not None:
+      # Make a copy of the shared http instance before we modify it.
+      http = copy.copy(http)
       http = credentials.authorize(http)
     if stats is not None:
       stats['duration'] = datetime.datetime.utcnow()