Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-2215

Order of filters in SecurityFilter.groovy doesn't match Acegi recommendations

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • _unsorted
    • None
    • Platform: All, OS: All

      The Acegi documentation here:

      http://www.acegisecurity.org/guide/springsecurity.html#filters

      specifies a clear ordering of filters. In particular, the documentation states
      that the RememberMeProcessingFilter should come after the
      AuthenticationProcessingFilter. In SecurityFilters.groovy, it looks like
      RememberMeProcessingFilter would be called before AuthenticationProcessingFilter2.

          [JENKINS-2215] Order of filters in SecurityFilter.groovy doesn't match Acegi recommendations

          Dean Yu created issue -

          Code changed in hudson
          User: : kohsuke
          Path:
          trunk/hudson/main/war/resources/WEB-INF/security/SecurityFilters.groovy
          trunk/www/changelog.html
          http://fisheye4.cenqua.com/changelog/hudson/?cs=11486
          Log:
          [FIXED JENKINS-2215] Changed the servlet filter ordering to match Acegi documentation.

          SCM/JIRA link daemon added a comment - Code changed in hudson User: : kohsuke Path: trunk/hudson/main/war/resources/WEB-INF/security/SecurityFilters.groovy trunk/www/changelog.html http://fisheye4.cenqua.com/changelog/hudson/?cs=11486 Log: [FIXED JENKINS-2215] Changed the servlet filter ordering to match Acegi documentation.
          SCM/JIRA link daemon made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]
          Andrew Bayer made changes -
          Status Original: Resolved [ 5 ] New: Closed [ 6 ]
          R. Tyler Croy made changes -
          Workflow Original: JNJira [ 132288 ] New: JNJira + In-Review [ 201295 ]
          Jenkins IRC Bot made changes -
          Component/s New: _unsorted [ 19622 ]
          Component/s Original: security [ 15508 ]

            Unassigned Unassigned
            dty Dean Yu
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

              Created:
              Updated:
              Resolved: