Merge pull request #3356 from d-giles/3276-longURI

URL Length check for locate_data

Author: bsipocz

CHANGES.rst

@@ -49,6 +49,7 @@ heasarc
 - Add support for astropy.table.Row in Heasarc.download_data and Heasarc.locate_data. [#3270]
 - Heasarc.locate_data returns empty rows with an error in the error_message column if there are
   no data associated with that row rather than filtering it out. [#3275]
+- Heasarc.locate_data changed to use POST request instead of GET to accomodate large requests. [#3356]
 
 imcce
 ^^^^^

astroquery/heasarc/core.py

@@ -517,13 +517,18 @@ def locate_data(self, query_result=None, catalog_name=None):
 
         # datalink url
         dlink_url = f'{self.VO_URL}/datalink/{catalog_name}'
-
         query = pyvo.dal.adhoc.DatalinkQuery(
             baseurl=dlink_url,
             id=query_result['__row'],
             session=self._session
         )
-        dl_result = query.execute().to_table()
+
+        dl_result = pyvo.dal.DALResults(
+            query.execute_votable(post=True),
+            url=query.queryurl,
+            session=query._session
+        ).to_table()
+
         # include rows that have directory links (i.e. data) and those
         # that report errors (usually means there are no data products)
         dl_result = dl_result[np.ma.mask_or(