Uploaded image for project: 'Magnolia Password Manager Module'
  1. Magnolia Password Manager Module
  2. PSWDMNGR-14

PasswordRegistry is not included as a component in the module descriptor

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Neutral
    • Resolution: Obsolete
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      PasswordRegistry class is not declared as a component in the module descriptor and therefore it can not be injected in certain cases like in subapp instantiation.

      Steps to reproduce in HelloWorld app

      1. Add maven dependency to Password-manager module
      2. Add PasswordRegistry parameter to the constructor:
        public HelloWorldMainSubApp(final SubAppContext subAppContext, HelloWorldMainSubAppView view, LocationController locationController, PasswordRegistry passwordRegistry) 
      3. Deploy and access hello world app you will get:
        info.magnolia.objectfactory.MgnlInstantiationException: Unable to resolve parameters for constructor public info.magnolia.training.developer.admincentral.vaadin.apps.helloworld.HelloWorldMainSubApp(info.magnolia.ui.api.app.SubAppContext,info.magnolia.training.developer.admincentral.vaadin.apps.helloworld.HelloWorldMainSubAppView,info.magnolia.ui.api.location.LocationController,info.magnolia.keystore.registry.PasswordRegistry). Unresolved parameter(s) are: 4th parameter which is of type info.magnolia.keystore.registry.PasswordRegistry

      This is not happening in every case PasswordRegistry is injected, for instance it is successfully injected in info.magnolia.ai.image.amazon.AmazonImageRecogniser.

      PSWDMNGR-13 should be taken into account in terms of security

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                ajuran AntonĂ­n Juran
                Reporter:
                jayala Jonathan Ayala
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Date of First Response: