CAS StorageStorageService configuration

[Walter Forbes Hoehn (wassa)]

I just added a JIRA issue for this, but was wondering if anyone would object to adding a property for configuring the CAS StorageService.  A proposed patch is below.  This change would allow the CAS ST/PT storage to be configured in a way that is similar to how other IdP components define their StorageService implementations.  I know that this could be done currently by adding an override in conf/cas-protocol.xml , but I found that a bit confusing because a) there is currently no similar configuration information in that file and b) other StorageService implementations are specified in idp.properties .

Thoughts?

-WFH


Index: cas-protocol-system.xml
===================================================================
--- cas-protocol-system.xml   (revision 7369)
+++ cas-protocol-system.xml   (working copy)
@@ -11,9 +11,9 @@
        default-destroy-method="destroy">

     <bean id="cas.ticketService" class="net.shibboleth.idp.cas.ticket.SimpleTicketService"
-          c:service-ref="shibboleth.StorageService" />
+          c:service-ref="%{idp.cas.StorageService:shibboleth.StorageService}" />

     <bean id="cas.proxyAuthenticator" class="net.shibboleth.idp.cas.proxy.HttpClientProxyAuthenticator" />

     <import resource="../../conf/cas-protocol.xml" />
-</beans>
\ No newline at end of file
+</beans>