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

gelf: support HTTP authentication #9355

Open
arvidvillen opened this issue May 15, 2020 · 5 comments
Open

gelf: support HTTP authentication #9355

arvidvillen opened this issue May 15, 2020 · 5 comments
Labels
area/logging kind/enhancement New feature or request

Comments

@arvidvillen
Copy link

Description
When configuring Amazon Elasticsearch Service with public access and you don't want to configure IP based security it is recommended to use HTTP authentication and it would be nice if quarkus-logging-gelf supported that.

Implementation ideas
Even if its not very clear in the documentation it seems that the extension supports HTTPS by setting quarkus.log.handler.gelf.host to something like https://example.com/, however adding the credentials like https://username:[email protected]/ does not work.

Additional note
See mp911de/logstash-gelf#194

@arvidvillen arvidvillen added the kind/enhancement New feature or request label May 15, 2020
@loicmathieu loicmathieu added area/logging triage/on-ice Frozen until external concerns are resolved labels May 18, 2020
@loicmathieu
Copy link
Contributor

@arvidvillen the logstash-gelf library didn't support HTTP authentication yet so I put this enhancement on ice untill it supports basic auth, then we can have a look at supporting it (and documenting the HTTP support).

@gsmet
Copy link
Member

gsmet commented May 20, 2020

Has a feature request been open for logstash-gelf?

@loicmathieu
Copy link
Contributor

Yes, it is already linked in the description and apparently @arvidvillen is working on it so as soon as it's released we can un-freeze this one and implement it.

@loicmathieu
Copy link
Contributor

The enhancement has been done on the upstream library, it will be available on 1.15.0 wich is not yet publshed.

@loicmathieu
Copy link
Contributor

Hi @arvidvillen the new version of logstash-gelf has been integrated in Quarkus and support HTTP authentication.

Will you still willing to implement this for Quarkus ?

@loicmathieu loicmathieu removed the triage/on-ice Frozen until external concerns are resolved label Jan 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/logging kind/enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants