Skip to content
This repository has been archived by the owner on Jun 29, 2023. It is now read-only.

Basic authentication to graylog server #194

Closed
chka73 opened this issue Apr 9, 2019 · 11 comments · Fixed by #254
Closed

Basic authentication to graylog server #194

chka73 opened this issue Apr 9, 2019 · 11 comments · Fixed by #254
Labels
type: enhancement A general enhancement

Comments

@chka73
Copy link

chka73 commented Apr 9, 2019

Version 1.13.0

Documentation request/ feature request.

Looking for information about if I can configure the Gelf appender to use basic authentication to our Graylog server?

@mp911de
Copy link
Owner

mp911de commented Apr 9, 2019

Have you tried adding credentials to the HTTP url (http://user:pass@server)?

@mp911de mp911de added the status: waiting-for-triage An issue we've not yet triaged label Apr 9, 2019
@chka73
Copy link
Author

chka73 commented Apr 9, 2019

Yes. Tried that as first choice. Getting a 401 response from server though so my guess was that the user part of the URL was stripped in some step?

@mp911de
Copy link
Owner

mp911de commented Apr 10, 2019

We had once a ticket #150 to add basic auth, but we never followed up on that one. So currently, there is no basic authentication support.

@mp911de mp911de added type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged labels Jun 13, 2019
@cchet
Copy link
Contributor

cchet commented Jun 26, 2019

@mp911de
I would like to implement that

@mp911de
Copy link
Owner

mp911de commented Jun 26, 2019

Feel free to give it a spin. Make sure to stay Java 6-compatible.

@arvidvillen
Copy link

Any updates on this?

@mp911de
Copy link
Owner

mp911de commented May 15, 2020

Pull requests are welcome!

@arvidvillen
Copy link

Hi,
Is java 6 still a requirement?
Do you think DatatypeConverter could be used for base64 encoding?

@mp911de
Copy link
Owner

mp911de commented May 15, 2020

That one comes from the JAXB package. Currently, the baseline is Java 1.7. It’s fine if we ship our own Base64 implementation as package-private utility. Ideally, we an find a source that is as well under MIT license to not violate any license terms.

@arvidvillen
Copy link

@mp911de
Copy link
Owner

mp911de commented Jun 8, 2020

The referenced code should be fine. Feel free to come up with a PR. We should keep the Base64 support package-private and make sure to include the full license header to preserve its origin.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type: enhancement A general enhancement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants