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.

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

JSON output standard format #109

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

JSON output standard format #109

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

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