Move check_hasreginfo to eduGAIN import policy

Move this check from a location where its effects may be discarded into the general policy checks so that results are always visible. See ukf/ukf-meta#210.
InCommon · Dec 29, 2019 · 85407ee · 85407ee
1 parent 42bf733
commit 85407ee
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 6 deletions.
diff --git a/mdx/int_edugain/beans.xml b/mdx/int_edugain/beans.xml
@@ -94,12 +94,6 @@
                 <ref bean="disassemble"/>
 
                 <ref bean="int_edugain_removeBlacklistedEntities"/>
-
-                <!--
-                    All eduGAIN entities should have mdrpi:RegistrationInfo elements, but
-                    we can't check the actual values.
-                -->
-                <ref bean="check_hasreginfo"/>
             </list>
         </property>
     </bean>

diff --git a/mdx/uk/edugain-policy.xml b/mdx/uk/edugain-policy.xml
@@ -121,6 +121,25 @@
                     </property>
                 </bean>
 
+                <!--
+                    ***************************************************
+                    ***                                             ***
+                    ***   U K   F E D E R A T I O N   P O L I C Y   ***
+                    ***                                             ***
+                    ***************************************************
+
+                    The following policy elements are not documented as part
+                    of InCommon's eduGAIN import policy. Some may be
+                    _implemented_ as part of eduGAIN's effective policy, but
+                    not documented as such.
+                -->
+
+                <!--
+                    All eduGAIN entities should have mdrpi:RegistrationInfo elements, but
+                    we can't check the actual values.
+                -->
+                <ref bean="check_hasreginfo"/>
+
             </list>
         </property>
     </bean>