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

json formatter support backslash unescape #5

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

Conversation

sunfuze
Copy link

@sunfuze sunfuze commented Jul 8, 2022

support backslash unescape

@vercel
Copy link

vercel bot commented Jul 8, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
bazinga ✅ Ready (Inspect) Visit Preview Jul 8, 2022 at 5:56AM (UTC)

@luin
Copy link
Owner

luin commented Jul 9, 2022

Hey @sunfuze 👋 ,

Thanks for the pull request! I'm wondering what's the main purpose/use case of this change?

@sunfuze
Copy link
Author

sunfuze commented Jul 10, 2022

We use Archery as database manager, it append backslash when field is json.

Example:
image

So it is usable in this case to remove backslash when format json content.

@sunfuze sunfuze closed this Jul 10, 2022
@sunfuze sunfuze reopened this Jul 10, 2022
@sunfuze
Copy link
Author

sunfuze commented Jul 10, 2022

Another case is: our log system use json as output format, if some log field is json string, the output of that field will be appended backslash before '"'.

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.

2 participants