Replies: 1 comment
-
The suggestion is implemented and submitted as a pull request #50 Potential improvements:
|
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
LookupClient class in peppol-lookup is quite flexible - it is possible to define in configuration custom implementations of each its part:
But it does not provide a way to overwrite the whole implementation of LookupClient, e.g. for testing purposes.
All mentioned parts are configurable via
mode.default.lookup.PART.class
- except LookupClient implementation, see https://github.com/OxalisCommunity/vefa-peppol/blob/56ab2e5a128081f733794ae69493b480c68bc34b/peppol-lookup/src/main/resources/reference.confIn this discussion it is proposed to expand configuration with
mode.default.lookup.impl.class = network.oxalis.vefa.peppol.lookup.LookupClient
and make LookupClient instantiation dynamic in LookupClientBuilder (see last 4 lines below):and provide an example of DummyLookupClient implementation and configuration, which can be overwritten via reference.conf at application class path e.g. like this:
Theoretically, it can be also used for a "standalone" version, to omit multiple parameters in command line, but definition of the "static" lookup result via reference.conf.
Beta Was this translation helpful? Give feedback.
All reactions