[jboss-jira] [JBoss JIRA] (WFLY-8526) Confusion in method with(AuthenticationContext other) in AuthenticationContext

[Darran Lofthouse (JIRA)]

     [ https://issues.jboss.org/browse/WFLY-8526?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Darran Lofthouse updated WFLY-8526:
-----------------------------------
    Fix Version/s: 11.0.0.Beta1


> Confusion in method with(AuthenticationContext other) in AuthenticationContext
> ------------------------------------------------------------------------------
>
>                 Key: WFLY-8526
>                 URL: https://issues.jboss.org/browse/WFLY-8526
>             Project: WildFly
>          Issue Type: Bug
>          Components: Security
>            Reporter: Darran Lofthouse
>            Assignee: Jan Kalina
>            Priority: Blocker
>              Labels: api, authentication-context
>             Fix For: 11.0.0.Beta1
>
>
> org.wildfly.security.auth.client.AuthenticationContext includes method {{AuthenticationContext with(AuthenticationContext other)}} which creates new AuthenticationContext which includes rules and configuration and SSL context of given AuthenticationContext other.
> However, in case when {{with}} method is used with index and another AuthenticationContext, then it includes only rules and configuration (SSL context is not used). There is also method {{withSsl}} which includes rules and SSL context, but no configuration.
> I see three problems here:
> * there is different behavior between {{with(AuthenticationContext other)}} and {{with(int idx, AuthenticationContext other)}} - first includes also SSL context
> * javadoc for with(AuthenticationContext other) does not describe that SSL context from given {{AuthenticationContext other}} is also used.
> * there is not able to include both configuration and SSL context into any AuthenticationContext on some position based on index
> I report this as critical because it is part of public API - it should stay backward compatible once it will be released.



--
This message was sent by Atlassian JIRA
(v7.2.3#72005)