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

Add various Base encoding functions #190

Merged
merged 1 commit into from Sep 11, 2017
Merged

Add various Base encoding functions #190

merged 1 commit into from Sep 11, 2017

Conversation

@joschi
Copy link
Contributor

@joschi joschi commented Jun 20, 2017

This PR adds various Base encoding functions (Base16, Base32, Base64, and some variants).

In fact, it provides the functionality of Guava's BaseEncoding to the processing pipelines.

This can be useful to transport binary data over the wire in a safe-ish format.

Discussion from IRC:

<joschi> jose_: could you elaborate on the use case, please?
<jose_> so briefly i'm at fastly, we're using a lot of graylog internally (and wrote a blog post series on it). i'm doing a demo next week of our WAF product and using graylog as a log analysis platform for the workshop.
<jose_> as part of our logging we export attack data (from the client) base64 encoded to avoid fouling up log parsers (it's CSV records streamed out)
<jose_> rather than trying to escape all possible screwyness
<jose_> so after it arrives and then extracted using a CSV field split, and before it gets stored, i'd like to un-encode it
<jose_> so i can see it natively in the product
@joschi joschi requested a review from kroepke Jun 20, 2017
@bernd bernd added this to the 2.4.0 milestone Jul 18, 2017
@bernd bernd self-assigned this Sep 11, 2017
@bernd
bernd approved these changes Sep 11, 2017
Copy link
Member

@bernd bernd left a comment

LGTM 👍

@bernd bernd merged commit 125a754 into master Sep 11, 2017
4 checks passed
4 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@garybot2
graylog-project/pr Jenkins build graylog-project-pr-snapshot 247 has succeeded
Details
license/cla Contributor License Agreement is signed.
Details
@bernd bernd deleted the baseX-codec branch Sep 11, 2017
kroepke added a commit that referenced this pull request Oct 19, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants