replace urllib with conda.gateways.conection Session

conda_env/env.py

@@ -8,12 +8,12 @@
 import os
 import re
 import json
-from conda._vendor.auxlib._vendor.six.moves import urllib
 
 from conda.base.context import context
 from conda.cli import common  # TODO: this should never have to import form conda.cli
 from conda.common.serialize import yaml_load_standard
 from conda.core.prefix_data import PrefixData
+from conda.gateways.connection.session import CondaSession
 from conda.models.enums import PackageType
 from conda.models.match_spec import MatchSpec
 from conda.models.prefix_graph import PrefixGraph
@@ -146,10 +146,9 @@ def from_yaml(yamlstr, **kwargs):
 
 def from_file(filename):
     if any(filename.startswith(prefix) for prefix in ("https://", "http://")):
-        try:
-            yamlstr = urllib.request.urlopen(filename).read()
-        except urllib.error.HTTPError:
-            raise exceptions.EnvironmentFileNotFound(filename)
+        response = CondaSession().get(filename)
+        response.raise_for_status()
+        yamlstr = response.text
     elif not os.path.exists(filename):
         raise exceptions.EnvironmentFileNotFound(filename)
     else:

tests/conda_env/test_env.py

@@ -7,6 +7,7 @@
 
 from conda.core.prefix_data import PrefixData
 from conda.base.context import conda_tests_ctxt_mgmt_def_pol
+from conda.gateways.connection import HTTPError
 from conda.models.match_spec import MatchSpec
 from conda.common.io import env_vars
 from conda.common.serialize import yaml_load
@@ -84,7 +85,7 @@ def test_http(self):
         assert e.dependencies == f.dependencies
 
     def test_http_raises(self):
-        with self.assertRaises(exceptions.EnvironmentFileNotFound):
+        with self.assertRaises(HTTPError):
             env.from_file("https://raw.githubusercontent.com/conda/conda/master/tests/conda_env/support/does-not-exist.yml")