Uploaded image for project: 'Periscope'
  1. Periscope
  2. MGNLPER-37

RestSearchResultSupplier doesn't really use RestClient

    XMLWordPrintable

Details

    • Improvement
    • Resolution: Obsolete
    • Neutral
    • None
    • None
    • None

    Description

      info.magnolia.rest.client.registry.RestClientRegistry is injected into RestSearchResultSupplier, but it's used only to get baseURL from info.magnolia.rest.client.RestClientDefinition.

      In such a case info.magnolia.periscope.search.rest.RestSearchResultSupplierDefinition#restClientName can be change to #baseURL should be enough.

      Or we should change RestSearchResultSupplier to do request via info.magnolia.rest.client.RestClient

      Checklists

        Acceptance criteria

        Attachments

          Issue Links

            Activity

              People

                Unassigned Unassigned
                mdivilek Milan Divilek
                Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved:

                  Checklists

                    Task DoD