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

RestSearchResultSupplier doesn't really use RestClient

XMLWordPrintable

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

      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

        Acceptance criteria

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

                Created:
                Updated:
                Resolved: