Skip to content

JSON output standard format #109

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.

Sign up for GitHub

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

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

JSON output standard format #109

quentingosset wants to merge 1 commit into from

Conversation

@quentingosset
Copy link

@quentingosset quentingosset commented Feb 12, 2024 •
edited
Loading

Q A
Bug fix? no
New feature? JSON output standard format
Deprecations? no
Tickets #107
License MIT

So as not to break the logic of the parse that searches for the different terms in the whois. I've simply modified the output so that it respects the JSON standard.
Now :

  • Object names respect the CamelCase
  • Values are now enclosed in double quotes.
  • The return is a list containing the various object whois.

Current result :
image

Old result :
image

@whoisyulu
Copy link

whoisyulu commented Apr 11, 2024

This definitely looks better, but probably not "backward compatible" with existing applications.

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@quentingosset @whoisyulu