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

implement simple del function on object #43

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

cyriltovena
Copy link

Hey @jamesls,

Thank you for the great project, I was blown away by how clean this was implemented !

This PR introduces a way to delete multiple keys from an object, I know the PR is probably not semantically something we want for this language but I wanted it to be a starting point for a discussion on how we could achieve this.

We used jmespath in Grafana Loki to transform and select json, we love it and so far there is no better alternative. Though we would like to be able to delete key from objects because some logs may contains sensitive data or simply are not required. A bit like with jq del(.foo).

There is may be a better way to achieve this or may be we don't want this at all ?

I'm happy to contribute to the project to solve this issue, if we can settle on a design.

Again, Thank you

@suntong
Copy link

suntong commented Dec 24, 2021

I need this badly as well.

Please consider @jamesls

@springcomp
Copy link

@suntong @cyriltovena this is discussed here as part of our ideas for the future of JMESPath.

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

Successfully merging this pull request may close these issues.

3 participants