Flake8 formatting

Better line lengths for flake8 formatting. No change in
functionality.

Author: skoranda

src/satosa/micro_services/ldap_attribute_store.py

@@ -399,6 +399,8 @@ def process(self, context, data):
         Default interface for microservices. Process the input data for
         the input context.
         """
+        session_id = lu.get_session_id(context.state)
+
         issuer = data.auth_info.issuer
         requester = data.requester
         config = self.config.get(requester) or self.config["default"]
@@ -408,13 +410,13 @@ def process(self, context, data):
             "issuer": issuer,
             "config": self._filter_config(config),
         }
-        logline = lu.LOG_FMT.format(id=lu.get_session_id(context.state), message=msg)
+        logline = lu.LOG_FMT.format(id=session_id, message=msg)
         logger.debug(logline)
 
         # Ignore this SP entirely if so configured.
         if config["ignore"]:
             msg = "Ignoring SP {}".format(requester)
-            logline = lu.LOG_FMT.format(id=lu.get_session_id(context.state), message=msg)
+            logline = lu.LOG_FMT.format(id=session_id, message=msg)
             logger.info(logline)
             return super().process(context, data)
 
@@ -439,7 +441,7 @@ def process(self, context, data):
             if filter_value
         ]
         msg = {"message": "Search filters", "filter_values": filter_values}
-        logline = lu.LOG_FMT.format(id=lu.get_session_id(context.state), message=msg)
+        logline = lu.LOG_FMT.format(id=session_id, message=msg)
         logger.debug(logline)
 
         # Initialize an empty LDAP record. The first LDAP record found using
@@ -453,7 +455,7 @@ def process(self, context, data):
             "message": "LDAP server host",
             "server host": connection.server.host,
         }
-        logline = lu.LOG_FMT.format(id=lu.get_session_id(context.state), message=msg)
+        logline = lu.LOG_FMT.format(id=session_id, message=msg)
         logger.debug(logline)
 
         for filter_val in filter_values:
@@ -463,7 +465,7 @@ def process(self, context, data):
                 "message": "LDAP query with constructed search filter",
                 "search filter": search_filter,
             }
-            logline = lu.LOG_FMT.format(id=lu.get_session_id(context.state), message=msg)
+            logline = lu.LOG_FMT.format(id=session_id, message=msg)
             logger.debug(logline)
 
             attributes = (
@@ -485,14 +487,14 @@ def process(self, context, data):
                 exp_msg = "Caught unhandled exception: {}".format(err)
 
             if exp_msg:
-                logline = lu.LOG_FMT.format(id=lu.get_session_id(context.state), message=exp_msg)
+                logline = lu.LOG_FMT.format(id=session_id, message=exp_msg)
                 logger.error(logline)
                 return super().process(context, data)
 
             if not results:
                 msg = "Querying LDAP server: No results for {}."
                 msg = msg.format(filter_val)
-                logline = lu.LOG_FMT.format(id=lu.get_session_id(context.state), message=msg)
+                logline = lu.LOG_FMT.format(id=session_id, message=msg)
                 logger.debug(logline)
                 continue
 
@@ -502,10 +504,10 @@ def process(self, context, data):
                 responses = connection.get_response(results)[0]
 
             msg = "Done querying LDAP server"
-            logline = lu.LOG_FMT.format(id=lu.get_session_id(context.state), message=msg)
+            logline = lu.LOG_FMT.format(id=session_id, message=msg)
             logger.debug(logline)
             msg = "LDAP server returned {} records".format(len(responses))
-            logline = lu.LOG_FMT.format(id=lu.get_session_id(context.state), message=msg)
+            logline = lu.LOG_FMT.format(id=session_id, message=msg)
             logger.info(logline)
 
             # For now consider only the first record found (if any).
@@ -514,7 +516,7 @@ def process(self, context, data):
                     msg = "LDAP server returned {} records using search filter"
                     msg = msg + " value {}"
                     msg = msg.format(len(responses), filter_val)
-                    logline = lu.LOG_FMT.format(id=lu.get_session_id(context.state), message=msg)
+                    logline = lu.LOG_FMT.format(id=session_id, message=msg)
                     logger.warning(logline)
                 record = responses[0]
                 break
@@ -524,7 +526,7 @@ def process(self, context, data):
         if config["clear_input_attributes"]:
             msg = "Clearing values for these input attributes: {}"
             msg = msg.format(data.attributes)
-            logline = lu.LOG_FMT.format(id=lu.get_session_id(context.state), message=msg)
+            logline = lu.LOG_FMT.format(id=session_id, message=msg)
             logger.debug(logline)
             data.attributes = {}
 
@@ -549,7 +551,7 @@ def process(self, context, data):
                 "DN": record["dn"],
                 "attributes": record["attributes"],
             }
-            logline = lu.LOG_FMT.format(id=lu.get_session_id(context.state), message=msg)
+            logline = lu.LOG_FMT.format(id=session_id, message=msg)
             logger.debug(logline)
 
             # Populate attributes as configured.
@@ -573,11 +575,11 @@ def process(self, context, data):
             # may use it if required.
             context.decorate(KEY_FOUND_LDAP_RECORD, record)
             msg = "Added record {} to context".format(record)
-            logline = lu.LOG_FMT.format(id=lu.get_session_id(context.state), message=msg)
+            logline = lu.LOG_FMT.format(id=session_id, message=msg)
             logger.debug(logline)
         else:
             msg = "No record found in LDAP so no attributes will be added"
-            logline = lu.LOG_FMT.format(id=lu.get_session_id(context.state), message=msg)
+            logline = lu.LOG_FMT.format(id=session_id, message=msg)
             logger.warning(logline)
             on_ldap_search_result_empty = config["on_ldap_search_result_empty"]
             if on_ldap_search_result_empty:
@@ -592,11 +594,11 @@ def process(self, context, data):
                     encoded_idp_entity_id,
                 )
                 msg = "Redirecting to {}".format(url)
-                logline = lu.LOG_FMT.format(id=lu.get_session_id(context.state), message=msg)
+                logline = lu.LOG_FMT.format(id=session_id, message=msg)
                 logger.info(logline)
                 return Redirect(url)
 
         msg = "Returning data.attributes {}".format(data.attributes)
-        logline = lu.LOG_FMT.format(id=lu.get_session_id(context.state), message=msg)
+        logline = lu.LOG_FMT.format(id=session_id, message=msg)
         logger.debug(logline)
         return super().process(context, data)