tkuchiki / alp

Access Log Profiler
MIT License
670 stars 34 forks source link

`alp json --dump` does not work 1.0.13 rel #74

Closed chenrui333 closed 1 year ago

chenrui333 commented 1 year ago

👋 while upgrading alp to the latest release, 1.0.13, I ran into some regression test failure issue (--dump stops working for alp json), thanks!

error log as below:

==> /opt/homebrew/Cellar/alp/1.0.13/bin/alp json --file=/private/tmp/alp-test-20230722-91695-180mec3/json_access.log --dump=/private/tmp/alp-test-20230722-91695-180mec3/dump.yml
+-------+-----+-----+-----+-----+-----+--------+-------------------+--------+--------+--------+--------+--------+--------+--------+--------+-----------+-----------+-----------+-----------+
| COUNT | 1XX | 2XX | 3XX | 4XX | 5XX | METHOD |        URI        |  MIN   |  MAX   |  SUM   |  AVG   |  P90   |  P95   |  P99   | STDDEV | MIN(BODY) | MAX(BODY) | SUM(BODY) | AVG(BODY) |
+-------+-----+-----+-----+-----+-----+--------+-------------------+--------+--------+--------+--------+--------+--------+--------+--------+-----------+-----------+-----------+-----------+
| 1     | 0   | 1   | 0   | 0   | 0   | POST   | /hoge/piyo        | 0.234  | 0.234  | 0.234  | 0.234  | 0.234  | 0.234  | 0.234  | 0.000  | 34.000    | 34.000    | 34.000    | 34.000    |
| 1     | 0   | 1   | 0   | 0   | 0   | GET    | /diary/entry/1234 | 0.135  | 0.135  | 0.135  | 0.135  | 0.135  | 0.135  | 0.135  | 0.000  | 15.000    | 15.000    | 15.000    | 15.000    |
| 1     | 0   | 1   | 0   | 0   | 0   | GET    | /diary/entry/5678 | 0.432  | 0.432  | 0.432  | 0.432  | 0.432  | 0.432  | 0.432  | 0.000  | 30.000    | 30.000    | 30.000    | 30.000    |
| 1     | 0   | 0   | 0   | 0   | 1   | GET    | /foo/bar/5xx      | 60.000 | 60.000 | 60.000 | 60.000 | 60.000 | 60.000 | 60.000 | 0.000  | 15.000    | 15.000    | 15.000    | 15.000    |
| 1     | 0   | 1   | 0   | 0   | 0   | GET    | /req              | 0.321  | 0.321  | 0.321  | 0.321  | 0.321  | 0.321  | 0.321  | 0.000  | 15.000    | 15.000    | 15.000    | 15.000    |
| 2     | 0   | 2   | 0   | 0   | 0   | GET    | /foo/bar          | 0.123  | 0.123  | 0.246  | 0.123  | 0.123  | 0.123  | 0.123  | 0.000  | 56.000    | 56.000    | 112.000   | 56.000    |
| 5     | 0   | 5   | 0   | 0   | 0   | POST   | /foo/bar          | 0.057  | 0.234  | 0.548  | 0.110  | 0.234  | 0.234  | 0.234  | 0.065  | 12.000    | 34.000    | 126.000   | 25.200    |
+-------+-----+-----+-----+-----+-----+--------+-------------------+--------+--------+--------+--------+--------+--------+--------+--------+-----------+-----------+-----------+-----------+

let me know if I need to provide more info.

relates to https://github.com/Homebrew/homebrew-core/pull/137306

tkuchiki commented 1 year ago

@chenrui333 Thank you for opening issue! I will check it.

tkuchiki commented 1 year ago

I'm working on releasing v1.0.14 now. Just a moment, please.

tkuchiki commented 1 year ago

🎉 https://github.com/tkuchiki/alp/releases/tag/v1.0.14

chenrui333 commented 1 year ago

1.0.14 works for me, thanks for the fix!