]
Yeray Borges reassigned WFLY-8997:
----------------------------------
Assignee: Yeray Borges (was: Stefan Guilhen)
@RunAsIdentity should cause authentication part to be skipped
-------------------------------------------------------------
Key: WFLY-8997
URL:
https://issues.jboss.org/browse/WFLY-8997
Project: WildFly
Issue Type: Bug
Components: Security
Affects Versions: 11.0.0.Alpha1
Reporter: Jörg Bäsner
Assignee: Yeray Borges
The issue [
WFLY-140|https://issues.jboss.org/browse/WFLY-140] introduced a change in
behavior.
Before this change, the SecurityContextInterceptor would just invoke the push() method on
SimpleSecurityManager and that method would internally create a new security context and
authenticate the incoming principal if needed. In that implementation the presence of a
RunAsIdentity would cause authentication part to be skipped.
With the changes in the above issue, the security context establishment and the
authentication parts were separated and while push() still checks for a RunAsIdentity, the
authenticate() implementation does not, which ends up triggering the authentication
process even if a RunAsIdentity is available. There is another check in place to avoid
authentication if a valid authenticated subject already exists and the security domains
match but this should also be the case if the security domains do not match.