Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

No way to configure HEC HttpClient to use proxy #117

Open
andy-paine opened this issue May 8, 2019 · 3 comments
Open

No way to configure HEC HttpClient to use proxy #117

andy-paine opened this issue May 8, 2019 · 3 comments

Comments

@andy-paine
Copy link

andy-paine commented May 8, 2019

Version of this project you're using: 1.6.2
Platform version: Debian Stretch
Framework version: JRE 11
Splunk version: 7.0.8.5
Other relevant information:

We are using the HttpEventCollectorLogbackAppender to send some of our logs to splunkcloud. The app sending these logs lives behind an egress proxy (for allowing only certain domains etc.)
The HttpEventCollectorSender class news up an Apache HttpClient using the HttpAsyncClients.custom() method.

Unfortunately the Apache HttpClient class does not respect the https.proxyHost/http.proxyPort system properties. As there is no way for us to new up the client ourselves and there are no proxy settings that we can pass in to the HttpEventCollectorLogbackAppender, we cannot get the appender to reach splunkcloud through the egress proxy.

It would be great if either the HttpClient could use the system properties (HttpAsyncClients.custom().useSystemProperties()) or if the proxy settings could be passed into the HttpEventCollectorLogbackAppender constructor so that we can correctly configure the client.

@shakeelmohamed
Copy link
Contributor

Hi @andy-paine,

Thanks for creating an issue. We're open to a pull request for making the changes to support your use case, I think this is a reasonable change to bring in.

@kfud
Copy link

kfud commented Jul 9, 2019

I agree with this one, we also needed the useSystemProperties(). A protected method would have been nice to at least override or enrich the HTTPAsyncClients settings. I could throw together a quick pull request if @andy-paine hasn't already started one.

kfud pushed a commit to fudcom/splunk-library-javalogging that referenced this issue Jul 10, 2019
@rmozone
Copy link

rmozone commented Apr 18, 2023

Hello! Any news regarding this issue?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants