Closed barend closed 4 years ago
I renamed RequestFlowExtension
to RequestInterceptor
, which I think is a little easier on the eyes. This is slightly inaccurate, because it intercepts not just requests but also responses. However, I think people will figure that out quickly enough when the compiler tells them to implement the interceptResponse
method.
I renamed
RequestFlowExtension
toRequestInterceptor
, ...
There are still some traces of classes named Xxx(flow)Extension. Is that on purpose?
There are still some traces of classes named Xxx(flow)Extension. Is that on purpose?
@mmartina No, that was entirely by mistake. I have addressed this.
I have now (also) added the auth config to the documentation.
Fixes #66 .
This PR adds an interceptor API for hooking an extension into the HTTP traffic between Scruid and Druid. The API allows the simple case of intercepting the request before its transmission (as implemented by
BasicAuthenticationExtension
), as well as the complicated case of intercepting the response, reading an authentication challenge, reaching out to some other endpoint to obtain a credential, and finally re-issuing the request.Implementation notes:
logback-classic
toProvided
, so that Scruid doesn't push it onto downstream projects as a transitive dependency.