Uploaded image for project: 'Apache Knox'
  1. Apache Knox
  2. KNOX-1651

Cleanup *IdentityAsserterMessages interfaces

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Blocker
    • Resolution: Fixed
    • None
    • 1.2.0
    • None
    • None

    Description

      Currently there are multiple IdentityAsserterMessages interfaces defined in two different modules. 

      gateway-provider-identity-assertion-common/src/main/java/org/apache/knox/gateway/IdentityAsserterMessages.java:public interface IdentityAsserterMessages {
      gateway-provider-identity-assertion-pseudo/src/main/java/org/apache/knox/gateway/IdentityAsserterMessages.java:public interface IdentityAsserterMessages {
      

      This doesn't cause any issues during mvn testing since each Maven module gets a different classpath. This came up when misconfiguring Knoxsso and got the following stacktrace.

      Caused by: java.lang.NoSuchMethodError: org.apache.knox.gateway.IdentityAsserterMessages.subjectNotAvailable()V
      at org.apache.knox.gateway.identityasserter.common.filter.CommonIdentityAssertionFilter.doFilter(CommonIdentityAssertionFilter.java:78)
      at org.apache.knox.gateway.GatewayFilter$Holder.doFilter(GatewayFilter.java:369)
      at org.apache.knox.gateway.GatewayFilter$Chain.doFilter(GatewayFilter.java:269)
      at org.apache.knox.gateway.webappsec.filter.XFrameOptionsFilter.doFilter(XFrameOptionsFilter.java:52)
      at org.apache.knox.gateway.GatewayFilter$Holder.doFilter(GatewayFilter.java:369)
      at org.apache.knox.gateway.GatewayFilter$Chain.doFilter(GatewayFilter.java:269)
      at org.apache.knox.gateway.filter.XForwardedHeaderFilter.doFilter(XForwardedHeaderFilter.java:30)
      at org.apache.knox.gateway.filter.AbstractGatewayFilter.doFilter(AbstractGatewayFilter.java:58)

      Attachments

        Issue Links

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            krisden Kevin Risden
            krisden Kevin Risden
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment