http --> https

Author: juanmcloaiza

astroquery/eso/__init__.py

@@ -18,7 +18,7 @@ class Conf(_config.ConfigNamespace):
         "",
         'Optional default username for ESO archive.')
     tap_url = _config.ConfigItem(
-        "http://archive.eso.org/tap_obs",
+        "https://archive.eso.org/tap_obs",
         'URL for TAP queries.')
     tap_url_dev = _config.ConfigItem(
         os.environ['TAP_URL_DEV'],

astroquery/eso/core.py

@@ -253,12 +253,12 @@ def query_tap_service(self,
         except pyvo.dal.exceptions.DALQueryError as e:
             raise pyvo.dal.exceptions.DALQueryError(f"\n\n\
                             Error executing the following query:\n\n{query_str}\n\n\
-                            See examples here: http://archive.eso.org/tap_obs/examples\n\n") from e
+                            See examples here: https://archive.eso.org/tap_obs/examples\n\n") from e
         except Exception as e:
             raise RuntimeError(f"\n\n\
                             Unknown exception {e} while executing the\
                             following query: \n\n{query_str}\n\n\
-                            See examples here: http://archive.eso.org/tap_obs/examples\n\n") from e
+                            See examples here: https://archive.eso.org/tap_obs/examples\n\n") from e
 
         if len(table_to_return) < 1:
             warnings.warn("Query returned no results", NoResultsWarning)
@@ -443,7 +443,7 @@ def get_headers(self, product_ids, *, cache=True):
         result = []
         for dp_id in product_ids:
             response = self._request(
-                "GET", f"http://archive.eso.org/hdr?DpId={dp_id}",
+                "GET", f"https://archive.eso.org/hdr?DpId={dp_id}",
                 cache=cache)
             root = BeautifulSoup(response.content, 'html5lib')
             hdr = root.select('pre')[0].text
@@ -500,7 +500,8 @@ def _find_cached_file(filename: str) -> bool:
             files_to_check.append(filename.rsplit(".", 1)[0])
         for file in files_to_check:
             if os.path.exists(file):
-                EsoClass.log_info(f"Found cached file {file}")
+                logmsg = (f"Found cached file {file}")
+                log.info(logmsg)
                 return True
         return False
 

astroquery/eso/tests/test_eso.py

@@ -32,9 +32,9 @@ def data_path(filename):
         },
     'POST':
         {
-            'http://archive.eso.org/wdb/wdb/eso/eso_archive_main/query': 'main_sgra_query.tbl',
-            'http://archive.eso.org/wdb/wdb/eso/amber/query': 'amber_sgra_query.tbl',
-            'http://archive.eso.org/wdb/wdb/adp/phase3_main/query': 'vvv_sgra_survey_response.tbl',
+            'https://archive.eso.org/wdb/wdb/eso/eso_archive_main/query': 'main_sgra_query.tbl',
+            'https://archive.eso.org/wdb/wdb/eso/amber/query': 'amber_sgra_query.tbl',
+            'https://archive.eso.org/wdb/wdb/adp/phase3_main/query': 'vvv_sgra_survey_response.tbl',
         },
     'ADQL':
         {
@@ -211,7 +211,7 @@ def test_tap_url():
     if EsoClass.USE_DEV_TAP:
         assert url == "http://dfidev5.hq.eso.org:8123/tap_obs"
     else:
-        assert url == "http://archive.eso.org/tap_obs"
+        assert url == "https://archive.eso.org/tap_obs"
 
 
 def test_py2adql():

astroquery/eso/utils.py

@@ -62,7 +62,7 @@ def py2adql(table: str, columns: Union[List, str] = None,
             order_by: str = '', order_by_desc=True, top: int = None):
     """
     Return the adql string corresponding to the parameters passed
-    See adql examples at http://archive.eso.org/tap_obs/examples
+    See adql examples at https://archive.eso.org/tap_obs/examples
     """
     # validate ra, dec, radius
     if not are_coords_valid(ra, dec, radius):