Skip to content

Remove schema.json and the code that generates it#627

Merged
webmat merged 5 commits intoelastic:masterfrom
webmat:remove-json
Nov 26, 2019
Merged

Remove schema.json and the code that generates it#627
webmat merged 5 commits intoelastic:masterfrom
webmat:remove-json

Conversation

@webmat
Copy link
Copy Markdown
Contributor

@webmat webmat commented Nov 22, 2019

This file is no longer used to power the SIEM field browser.

@webmat webmat self-assigned this Nov 22, 2019
@webmat webmat added the cleanup label Nov 22, 2019
Copy link
Copy Markdown
Contributor

@ruflin ruflin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should be careful to remove such file as I would argue if someone else uses these, it is a breaking change. Not to the schema, but to our tooling.

@webmat
Copy link
Copy Markdown
Contributor Author

webmat commented Nov 25, 2019

Its creator didn't even end up using it 🤷‍♂

@ruflin
Copy link
Copy Markdown
Contributor

ruflin commented Nov 25, 2019

🤦‍♂ That doesn't mean someone else is using it. I'm good with this "breaking change" but lets at least have it in the changelog so users know.

@webmat
Copy link
Copy Markdown
Contributor Author

webmat commented Nov 25, 2019

Yeah, someone else may be using it.

If anyone needs this, please make yourselves heard, even if this PR has already been merged. We can bring it back if needed :-)

@webmat
Copy link
Copy Markdown
Contributor Author

webmat commented Nov 25, 2019

Move this changelog entry to the tooling section
@pete-otaqui-sky
Copy link
Copy Markdown

Hi - I'd like to ask for this to be put back in. We'd like to be able to validate ECS compatibility using AWS API Gateway, which depends on JSON Schema. Would you like a PR for putting it back?

@ebeahan
Copy link
Copy Markdown
Member

ebeahan commented Jul 7, 2020

Hi @pete-otaqui-sky. Currently ECS doesn't maintain a JSON format of the schema as part of the generated artifacts. However, the ecs_nested.yml file is intended to be consumed programmatically and YAML can be painlessly converted to a JSON format if required for your needs.

Please feel free to open a new issue if you'd like to discuss further or have additional questions.

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