Uploaded image for project: 'WildFly'
  1. WildFly
  2. WFLY-8526

Confusion in method with(AuthenticationContext other) in AuthenticationContext

    XMLWordPrintable

Details

    Description

      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.

      Attachments

        Issue Links

          Activity

            People

              darran.lofthouse@redhat.com Darran Lofthouse
              darran.lofthouse@redhat.com Darran Lofthouse
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: