Merge pull request #320 from italia/revert-279-sp_metadata

Revert "Refactor the SP metadata loading logic."

templates/home.html

@@ -10,14 +10,11 @@ <h3 class="border-bottom mt-2 mb-2 border-italia">Service Provider configurati</
       <tbody>
         {% for item in sp_list %}
             <tr>
-                <td>
-                    {{ item['entityID'] }}
-                    <small class="float-right">
-                        <mark>loaded from {{ item['location'] }}</mark>
-                    </small>
+                <td >
+                   {{ item['entityID'] }}
                 </td>
            </tr>
         {% endfor %}
       </tbody>
     </table>
-{% endblock %}
+{% endblock %}

testenv/server.py

@@ -284,7 +284,7 @@ def _handle_http_post(self, action):
 
     def _get_certificates_by_issuer(self, issuer):
         try:
-            return self._registry.load(issuer).certs()
+            return self._registry.get(issuer).certs()
         except KeyError:
             self._raise_error(
                 'entity ID {} non registrato, impossibile ricavare'
@@ -357,7 +357,7 @@ def users(self):
                 'primary_attributes': spid_main_fields,
                 'secondary_attributes': spid_secondary_fields,
                 'users': self.user_manager.all(),
-                'sp_list': self._registry.load_all().keys(),
+                'sp_list': self._registry.all(),
                 'can_add_user': can_add_user
             }
         )
@@ -393,9 +393,8 @@ def index(self):
             **{
                 'sp_list': [
                     {
-                        "entityID": entity_id,
-                        "location": sp_metadata.location,
-                    } for (entity_id, sp_metadata) in self._registry.load_all().items()
+                        "entityID": sp
+                    } for sp in self._registry.all()
                 ],
             }
         )
@@ -406,7 +405,7 @@ def get_destination(self, req, sp_id):
         acs_index = getattr(req, 'assertion_consumer_service_index', None)
         protocol_binding = getattr(req, 'protocol_binding', None)
         if acs_index is not None:
-            acss = self._registry.load(
+            acss = self._registry.get(
                 sp_id).assertion_consumer_service(index=acs_index)
             if acss:
                 destination = acss[0].get('Location')
@@ -591,7 +590,7 @@ def login(self):
                                 atcs_idx
                             )
                         )
-                        sp_metadata = self._registry.load(sp_id)
+                        sp_metadata = self._registry.get(sp_id)
                         required = []
                         optional = []
                         if atcs_idx and sp_metadata:
@@ -792,7 +791,7 @@ def continue_response(self):
     def _sp_single_logout_service(self, issuer_name):
         _slo = None
         try:
-            _slo = self._registry.load(issuer_name).single_logout_services[0]
+            _slo = self._registry.get(issuer_name).single_logout_services[0]
         except Exception:
             pass
         return _slo