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

              Assignee:
              dlofthouse Darran Lofthouse
              Reporter:
              dlofthouse Darran Lofthouse
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: