[JIRA] (JCOMOIDC-66) Review ProfileConfiguration interfaces and concrete classes

Philip Smart (Jira) jira at shibboleth.atlassian.net
Mon Mar 27 14:18:34 UTC 2023


Philip Smart ( https://shibboleth.atlassian.net/secure/ViewProfile.jspa?accountId=5afecdc1eb3ffa0c596652c4 ) *commented* on JCOMOIDC-66 ( https://shibboleth.atlassian.net/browse/JCOMOIDC-66?atlOrigin=eyJpIjoiMmJlYTgxNzZhYmVkNDFkM2JmOTlkMGU4NWQ1OGZlZmMiLCJwIjoiaiJ9 )

Re: Review ProfileConfiguration interfaces and concrete classes ( https://shibboleth.atlassian.net/browse/JCOMOIDC-66?atlOrigin=eyJpIjoiMmJlYTgxNzZhYmVkNDFkM2JmOTlkMGU4NWQ1OGZlZmMiLCJwIjoiaiJ9 )

I took another go at this, other than changes to test classes (that actually instantiate the concrete classes), I’ve now got to a point where the only change would be to the profile configuration XML e.g.

<bean id="OIDC.SSO" parent="AbstractOIDCSSOProfile" lazy-init="true"
       class="net.shibboleth.oidc.profile.config.OIDCAuthorizationConfiguration" 

to:

<bean id="OIDC.SSO" parent="AbstractOIDCSSOProfile" lazy-init="true"
       class="net.shibboleth.oidc.profile.config.impl.DefaultOIDCAuthorizationConfiguration" 

Similarly for the other profile configs. The interfaces should now replace all the old concrete classes (in addition to new interfaces) — although I need to dbl with some failing OP tests.

( https://shibboleth.atlassian.net/browse/JCOMOIDC-66#add-comment?atlOrigin=eyJpIjoiMmJlYTgxNzZhYmVkNDFkM2JmOTlkMGU4NWQ1OGZlZmMiLCJwIjoiaiJ9 ) Add Comment ( https://shibboleth.atlassian.net/browse/JCOMOIDC-66#add-comment?atlOrigin=eyJpIjoiMmJlYTgxNzZhYmVkNDFkM2JmOTlkMGU4NWQ1OGZlZmMiLCJwIjoiaiJ9 )

Get Jira notifications on your phone! Download the Jira Cloud app for Android ( https://play.google.com/store/apps/details?id=com.atlassian.android.jira.core&referrer=utm_source%3DNotificationLink%26utm_medium%3DEmail ) or iOS ( https://itunes.apple.com/app/apple-store/id1006972087?pt=696495&ct=EmailNotificationLink&mt=8 ) This message was sent by Atlassian Jira (v1001.0.0-SNAPSHOT#100219- sha1:0152477 )
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://shibboleth.net/pipermail/commits/attachments/20230327/7d4c7982/attachment-0001.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: jira-generated-image-static-comment-icon-eae8e2b8-9a64-4d13-898f-864ef14b5fe4
Type: image/png
Size: 1084 bytes
Desc: not available
URL: <http://shibboleth.net/pipermail/commits/attachments/20230327/7d4c7982/attachment-0003.png>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: jira-generated-image-static-footer-desktop-logo-4c2a76cd-ada1-4a36-8784-a4c67f9bf9ca
Type: image/png
Size: 10805 bytes
Desc: not available
URL: <http://shibboleth.net/pipermail/commits/attachments/20230327/7d4c7982/attachment-0004.png>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: jira-generated-image-avatar-38e8779a-a0a2-4721-824a-34b65b21f407
Type: image/png
Size: 484 bytes
Desc: not available
URL: <http://shibboleth.net/pipermail/commits/attachments/20230327/7d4c7982/attachment-0005.png>


More information about the commits mailing list