Reformat based on @c00kiemons5ter comments

Author: Ioannis Kakavas

src/satosa/base.py

@@ -103,13 +103,15 @@ def _auth_req_callback_func(self, context, internal_request):
         state = context.state
         state[STATE_KEY] = {"requester": internal_request.requester}
         # TODO consent module should manage any state it needs by itself
-        if consent.STATE_KEY in context.state:
-            context.state[consent.STATE_KEY]["filter"] = internal_request.approved_attributes or []
-            context.state[consent.STATE_KEY]["requester_name"] = internal_request.requester_name
-        else:
-            context.state[consent.STATE_KEY] = {"filter": internal_request.approved_attributes or [],
-                                                "requester_name": internal_request.requester_name}
-
+        try:
+            state_dict = context.state[consent.STATE_KEY]
+        except KeyError:
+            state_dict = context.state[consent.STATE_KEY] = {}
+        finally:
+            state_dict.update({
+                "filter": internal_request.approved_attributes or [],
+                "requester_name": internal_request.requester_name,
+            })
         satosa_logging(logger, logging.INFO,
                        "Requesting provider: {}".format(internal_request.requester), state)