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

Make base64 encoding optional #38

Closed
wants to merge 0 commits into from

Conversation

pascalgulikers
Copy link

Thanks for contributing to Logstash! If you haven't already signed our CLA, here's a handy link: https://www.elastic.co/contributor-agreement/

@cla-checker-service
Copy link

cla-checker-service bot commented Dec 14, 2021

💚 CLA has been signed

@pascalgulikers
Copy link
Author

Can you please merge this PR? It's just a minor change and it will help us a lot! Thanks in advance :)

@pascalgulikers
Copy link
Author

@robbavey @roaksoax Best wishes! Can you please merge this PR?

@davehagler
Copy link

Is this PR still being considered? It would be most helpful

@andsel
Copy link
Contributor

andsel commented Apr 15, 2022

Hi @pascalgulikers
I've force pushed one commit to this PR and, for a reason I don't know, it closed this PR.
I've moved the work to #39 and added you as Co-author to preserve your contribution.
Sorry for that, I think it's due to be on main branch instead of a separate PR branch.

@andsel
Copy link
Contributor

andsel commented Apr 28, 2022

Hi @pascalgulikers thank's for your contribution 👏 this work has been published with https://rubygems.org/gems/logstash-codec-avro/versions/3.4.0-java

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

Successfully merging this pull request may close these issues.

4 participants