Skip to content

Comments

Feature/validate xml#30

Open
dngferreira wants to merge 3 commits intologstash-plugins:mainfrom
dngferreira:feature/validate_xml
Open

Feature/validate xml#30
dngferreira wants to merge 3 commits intologstash-plugins:mainfrom
dngferreira:feature/validate_xml

Conversation

@dngferreira
Copy link

Hi,
I thought that adding a way to validate XML documents when we receive them would be an interesting feature but I need some guidelines on how it can be done.
It seems to be working and the tests pass.

Would apreciate some code analysis since it's my first contribution to logstash.

@wiibaa
Copy link
Contributor

wiibaa commented May 24, 2016

@dngferreira Thanks for contributing! Can you please sign the CLA with your committer email so we can proceed with your PR.

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