[MGNLPER-37] RestSearchResultSupplier doesn't really use RestClient Created: 19/Jul/18  Updated: 02/Apr/20  Resolved: 02/Apr/20

Status: Closed
Project: Periscope
Component/s: None
Affects Version/s: None
Fix Version/s: None

Type: Improvement Priority: Neutral
Reporter: Milan Divilek Assignee: Unassigned
Resolution: Obsolete Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
dependency
depends upon MGNLRESTCL-53 Declarative REST clients Closed
supersession
is superseded by MGNLPER-119 Deprecate or remove RestSearchResultS... Closed
Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Epic Link: Periscope improvements

 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


Generated at Mon Feb 12 10:28:06 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.