Shiro
  1. Shiro
  2. SHIRO-222

Register LifecycleBeanPostProcessor before ShiroFilterFactoryBean [patch included]

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.1.0
    • Fix Version/s: 1.2.0
    • Component/s: Integration: Spring
    • Labels:
      None

      Description

      LifecycleBeanPostProcessor needs to be registred before ShiroFilterFactoryBean because...

      ShiroFilterFactoryBean itself is a BeanPostProcessor, like LifecycleBeanPostProcessor too. So during AbstractApplicationContext.registerBeanPostProcessors() these two beans are created and added to nonOrderedPostProcessors, after then they get registered. At this execution point ShiroFilterFactoryBean is already created with its SecurityManager and defined Realms. So LifecycleBeanPostProcessor will not effect anymore on Realms init.

      http://shiro-user.582556.n2.nabble.com/Shiro-and-Spring-with-ShiroFilterFactoryBean-td5811127.html

        Issue Links

          Activity

          manthos created issue -
          manthos made changes -
          Field Original Value New Value
          Attachment LifecycleBeanPostProcessor.patch [ 12465935 ]
          manthos made changes -
          Summary Register LifecycleBeanPostProcessor before ShiroFilterFactoryBean Register LifecycleBeanPostProcessor before ShiroFilterFactoryBean [patch included]
          manthos made changes -
          Link This issue is related to SHIRO-223 [ SHIRO-223 ]
          Les Hazlewood made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Fix Version/s 1.2.0 [ 12315478 ]
          Resolution Fixed [ 1 ]
          Les Hazlewood made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

            People

            • Assignee:
              Unassigned
              Reporter:
              manthos
            • Votes:
              1 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development