Merge pull request #204 from skanct/ck_default_userinfo_request_method

Changes the default for the userinfo_request_method to GET

Author: c00kiemon5ter

src/satosa/backends/openid_connect.py

@@ -161,7 +161,7 @@ def _check_error_response(self, response, context):
             raise SATOSAAuthenticationError(context.state, "Access denied")
 
     def _get_userinfo(self, state, context):
-        kwargs = {"method": self.config["client"].get("userinfo_request_method", "POST")}
+        kwargs = {"method": self.config["client"].get("userinfo_request_method", "GET")}
         userinfo_resp = self.client.do_user_info_request(state=state, **kwargs)
         self._check_error_response(userinfo_resp, context)
         return userinfo_resp.to_dict()

tests/satosa/backends/test_openid_connect.py

@@ -119,7 +119,7 @@ def setup_token_endpoint(self, token_endpoint_url, userinfo, signing_key):
                       content_type="application/json")
 
     def setup_userinfo_endpoint(self, userinfo_endpoint_url, userinfo):
-        responses.add(responses.POST,
+        responses.add(responses.GET,
                       userinfo_endpoint_url,
                       body=json.dumps(userinfo),
                       status=200,