Merge pull request #1018 from EnMAP-Box/1017-location-browser-request…

…sexceptionsjsondecodeerror-expecting-value-line-1-column-1-char-0 resolved #1017
EnMAP-Box · Nov 7, 2024 · ee5854b · ee5854b
2 parents ef59afa + f570025
commit ee5854b
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 2 deletions.
diff --git a/enmapbox/eo4qapps/locationbrowserapp/locationbrowserdockwidget.py b/enmapbox/eo4qapps/locationbrowserapp/locationbrowserdockwidget.py
@@ -130,11 +130,12 @@ def onRequestNominatimClicked(self):
         url = 'https://nominatim.openstreetmap.org/search?q=' \
               f'{urllib.parse.quote(text)}' \
               '&limit=50&extratags=1&polygon_geojson=1&format=json'
-        nominatimResults = requests.get(url).json()
+        headers = {'User-Agent': 'EnMAP-Box QGIS Plugin ([email protected])'}  # Required user agent
+        nominatimResults = requests.get(url, headers=headers).json()
         # find additional results
         if len(nominatimResults) == 1:
             url += f'&exclude_place_ids={nominatimResults[0]["place_id"]}'
-            nominatimResults.extend(requests.get(url).json())
+            nominatimResults.extend(requests.get(url, headers=headers).json())
 
         self.mResult.mList.clear()
         item = QListWidgetItem('')

diff --git a/tests/enmap-box/enmapbox/eo4qapps/locationbrowserapp/__init__.py b/tests/enmap-box/enmapbox/eo4qapps/locationbrowserapp/__init__.py
diff --git a/tests/enmap-box/enmapbox/eo4qapps/locationbrowserapp/test_LocationBrowserApp.py b/tests/enmap-box/enmapbox/eo4qapps/locationbrowserapp/test_LocationBrowserApp.py
@@ -0,0 +1,10 @@
+import requests
+from enmapboxprocessing.testcase import TestCase
+
+class LocationBrowserApp(TestCase):
+
+    def test_nominatim(self):
+        url = 'https://nominatim.openstreetmap.org/search?q=berlin&limit=50&extratags=1&polygon_geojson=1&format=json'
+        headers = {"User-Agent": "EnMAP-Box QGIS Plugin ([email protected])"}  # Required user agent
+        nominatimResults = requests.get(url, headers=headers)
+        self.assertEqual(nominatimResults.status_code, 200)