search

emsearcy / fluent-plugin-gelf

Buffered fluentd output plugin to GELF (Graylog2)
Apache License 2.0
33 stars 57 forks source link

Switch to use YAJL for JSON dumping #8

Closed bchew closed 9 years ago

bchew commented 9 years ago

Was getting the following warning in the logs when testing out the plugin: [warn]: emit transaction failed: error_class=Encoding::UndefinedConversionError error="\"\\xC3\" from ASCII-8BIT to UTF-8"

This switch seems to have resolved the issue, creating a PR if you would like to have it merged back.

bchew commented 9 years ago

It seems I've done the PR from master instead of a separate branch, so the later commits have come in - shall close this now, let me know if you would still like the first commit and I'll recreate the PR.