Giters

Screenly / cli

Command Line Interface (CLI) and GitHub Actions workflow for Screenly.

Home Page:https://screenly.io

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Fix JSON output

vpetersson opened this issue · comments

commented

It appears as some of our parsing is wrong.

$ screenly asset list --json  | jq
parse error: Invalid numeric literal at line 1, column 14

Looking closer into this, the error comes from the following debug message:

$ screenly asset list --json | head -n 1
2022-12-09T15:07:14.949Z INFO  [screenly] [
commented

Ok, it's because of today's change. Renat asked to use rust's logging library. It prepends this output.

commented

Ok, it breaks the -json feature so we need to improve how we log then. I also don't agree with unnecessary logging like the message we generate here as it has zero utility.