Skip to content
This repository has been archived by the owner on May 31, 2021. It is now read-only.

Gelf and UTF-8 #41

Merged
merged 1 commit into from
Nov 4, 2014
Merged

Gelf and UTF-8 #41

merged 1 commit into from
Nov 4, 2014

Conversation

Athou
Copy link
Contributor

@Athou Athou commented Oct 30, 2014

The GELF spec (http://www.graylog2.org/resources/gelf/specification) specifies that strings should be encoded as UTF8, String.getBytes() uses the platform default encoding which is not always UTF-8. This patch fixes it by forcing the use of UTF-8.

The GELF spec (http://www.graylog2.org/resources/gelf/specification) specifies that strings should be encoded as UTF8, String.getBytes() uses the platform default encoding which is not always UTF-8. This patch fixes it by forcing the use of UTF-8.
Moocar added a commit that referenced this pull request Nov 4, 2014
@Moocar Moocar merged commit cdb67a1 into Moocar:master Nov 4, 2014
@Moocar
Copy link
Owner

Moocar commented Nov 4, 2014

Thanks!

@Athou
Copy link
Contributor Author

Athou commented Nov 4, 2014

Would it be possible for you to publish this fix to maven central please?

@Athou Athou deleted the patch-1 branch November 4, 2014 16:03
@Moocar
Copy link
Owner

Moocar commented Nov 4, 2014

Done. I just released 0.12. It should be in central in a couple of hours

@Athou
Copy link
Contributor Author

Athou commented Nov 4, 2014

Awesome, thanks!

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

Successfully merging this pull request may close these issues.

2 participants