]
David Lloyd reassigned ELY-987:
-------------------------------
Assignee: David Lloyd (was: Darran Lofthouse)
Confusion in method with(AuthenticationContext other) in
AuthenticationContext
------------------------------------------------------------------------------
Key: ELY-987
URL:
https://issues.jboss.org/browse/ELY-987
Project: WildFly Elytron
Issue Type: Bug
Affects Versions: 1.1.0.Beta28
Reporter: Ondrej Lukas
Assignee: David Lloyd
Priority: Critical
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.