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

PasswordRegistry is not included as a component in the module descriptor

    XMLWordPrintable

Details

    • Bug
    • Resolution: Obsolete
    • Neutral
    • None
    • None
    • None
    • 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

      Checklists

        Acceptance criteria

        Attachments

          Issue Links

            Activity

              People

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

                Dates

                  Created:
                  Updated:
                  Resolved:

                  Checklists

                    Bug DoR
                    Task DoD