Merge pull request #1605 from svahul/response_code_param_url

response code for parameter "url"
openoereb · Sep 22, 2022 · 0aac2d7 · 0aac2d7
2 parents d0ba6e3 + 71719a7
commit 0aac2d7
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/pyramid_oereb/core/views/webservice.py b/pyramid_oereb/core/views/webservice.py
@@ -6,7 +6,7 @@
 import io
 # import re
 
-from pyramid.httpexceptions import HTTPBadRequest, HTTPFound, HTTPInternalServerError, HTTPNoContent, \
+from pyramid.httpexceptions import HTTPBadRequest, HTTPSeeOther, HTTPInternalServerError, HTTPNoContent, \
     HTTPNotFound
 from pyramid.path import DottedNameResolver
 from shapely.geometry import Point
@@ -598,12 +598,12 @@ def __redirect_to_dynamic_client__(real_estate):
                 The found real estate.
 
         Returns:
-            pyramid.httpexceptions.HTTPFound: The redirect response.
+            pyramid.httpexceptions.HTTPSeeOther: The redirect response.
         """
         url = Config.get_extract_config().get('redirect')
         if url is None:
             raise HTTPInternalServerError('Missing configuration for redirect to dynamic client.')
-        return HTTPFound(url.format(**vars(real_estate)))
+        return HTTPSeeOther(url.format(**vars(real_estate)))
 
 
 class Parameter(object):

diff --git a/tests/core/webservice/test_getextractbyid.py b/tests/core/webservice/test_getextractbyid.py
@@ -4,7 +4,7 @@
 import pytest
 from unittest.mock import patch
 from jsonschema import Draft4Validator
-from pyramid.httpexceptions import HTTPBadRequest, HTTPFound, HTTPNoContent
+from pyramid.httpexceptions import HTTPBadRequest, HTTPSeeOther, HTTPNoContent
 
 from tests.mockrequest import MockRequest
 from pyramid_oereb.core.views.webservice import PlrWebservice
@@ -238,5 +238,5 @@ def test_format_url(real_estate_data):
     })
     service = PlrWebservice(request)
     response = service.get_extract_by_id()
-    assert isinstance(response, HTTPFound)
+    assert isinstance(response, HTTPSeeOther)
     assert response.location == 'https://geoview.bl.ch/oereb/?egrid=TEST'