vulsio / gost

Build a local copy of Security Tracker. Notify via E-mail/Slack if there is an update.
MIT License
133 stars 44 forks source link

feat(log) no progress when --log-json option #267

Closed sadayuki-matsuno closed 4 months ago

sadayuki-matsuno commented 4 months ago

If this Pull Request is work in progress, Add a prefix of “[WIP]” in the title.

What did you implement:

no progress when --log-json option

Type of change

Please delete options that are not relevant.

How Has This Been Tested?

before

$ gost fetch debian --log-json
{"lvl":"info","msg":"Initialize Database","t":"2024-07-04T16:16:05.960068788+09:00"}
{"lvl":"info","msg":"Fetched all CVEs from Debian","t":"2024-07-04T16:16:06.00184586+09:00"}
{"CVEs":39701,"lvl":"info","msg":"Fetched","t":"2024-07-04T16:16:10.583478895+09:00"}
{"db":"sqlite3","lvl":"info","msg":"Insert Debian CVEs into DB","t":"2024-07-04T16:16:10.583537978+09:00"}
39701 / 39701 [----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------] 100.00% 5215 p/s

after

$ gost fetch debian --log-json
{"lvl":"info","msg":"Initialize Database","t":"2024-07-04T16:16:45.01428875+09:00"}
{"lvl":"info","msg":"Fetched all CVEs from Debian","t":"2024-07-04T16:16:45.051610124+09:00"}
{"CVEs":39701,"lvl":"info","msg":"Fetched","t":"2024-07-04T16:16:49.510853905+09:00"}
{"db":"sqlite3","lvl":"info","msg":"Insert Debian CVEs into DB","t":"2024-07-04T16:16:49.510899176+09:00"}

$ gost fetch debian
INFO[07-04|16:17:04] Initialize Database 
INFO[07-04|16:17:04] Fetched all CVEs from Debian 
INFO[07-04|16:17:08] Fetched                                  CVEs=39701
INFO[07-04|16:17:08] Insert Debian CVEs into DB               db=sqlite3
39701 / 39701 [----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------] 100.00% 4837 p/s

Checklist:

You don't have to satisfy all of the following.

Is this ready for review?: YES

Reference