search

Epinova / Epinova.Elasticsearch

A search-plugin for Episerver CMS and Commerce
MIT License
29 stars 20 forks source link

Feature/injected messagehandler #17

Closed CosX closed 5 years ago

CosX commented 5 years ago

Kan du teste om dette fungerer?

lonnkvist commented 5 years ago

Ser bra ut :) IncludeMessageHandler gir dog inntrykk av at man kan legge til flere handlere. Slik det er implementert nå vil alltid den siste som er lagt til overskrive MessageHandlers.Handler. Foreslår å enten endre ordlyden litt eller faktisk legge til støtte for en collection med handlere og bruke HttpClientFactory.Create (HttpClient sin ctor støtter bare èn handler).

CosX commented 5 years ago

Godt innspill. Jeg trodde kanskje bare at httpclientfactory kun fantes i dotnet core, men ser at de har en metode i framework også.

CosX commented 5 years ago

HttpClientFactory var litt mer jobb enn jeg trodde. Må inkludere .NET Standard 2.0. Men jeg endret ordlyden litt og dokumenterte litt i samme slengen.