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

Fix to make oink skip lines with bad character encoding #7

Open
wants to merge 4 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@sgibbons

sgibbons commented Jan 26, 2011

Tweak to avoid crashing on non-UTF8 log lines. Simply skips processing for those lines.

@noahd1

This comment has been minimized.

Owner

noahd1 commented Feb 1, 2011

Thank you for this pull request. Any chance you could add a spec to cover this? I will merge with some test coverage. Thanks again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment