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 json output format #14

Open
ibizaman opened this issue Jan 20, 2025 · 0 comments
Open

Add json output format #14

ibizaman opened this issue Jan 20, 2025 · 0 comments

Comments

@ibizaman
Copy link

Thank you for this tool! I've been following it since you announced it in the PR in the lldap project.

I am working on integrating lldap-cli into NixOS. As you may know, NixOS is all about declarative setup, so I'm creating a script which runs lldap-cli command according to some static config to create/delete groups and users and memberships to groups.

The current output format of lldap-cli is made for users, with its tab separated format. For what I'm doing, this is really hard to parse. Instead, keeping the original json format coming out of graphql is much more valuable in this case. Would you be okay to add an option to change the output format to json?

I created a fork which simply changes the output format from tsv to json, without giving an option to choose. ibizaman@e07e2ca I won't create a PR with this but it gives an idea of the amount of work this will entail.
Btw, I'd be happy to create a proper PR but I'd like some guidance on how to integrate handling such an option in the script.

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

No branches or pull requests

1 participant