Add support to measure status_codes

[wpjunior]

#154

[u5surf]

@wpjunior Thanks contribute. Unfortunately, the status dash board(html page) seems to corrupt the series of N/A appered suddenly in server zone.

Note that the page is used by many users now a day, they will suprise what happened when the feature turn on.
We should be handling or treating something for all I/F thus html, json, prometheus to keep backward compatibility, even if you don't need that for your usecase.

[wpjunior]

Hi @u5surf, I extracted the status codes into a separate JSON structure, so the status page is still working as before.

I believe I have finished the merge request.

[u5surf]

@wpjunior
Hi, Could you handling following control command?

https://github.com/vozlt/nginx-module-vts?tab=readme-ov-file#to-reset-traffic-zones-on-the-fly

The status 1xx - 5xx can be reset by this command on the fly.
We should make a consistent with this behavior.

[wpjunior]

Hi @u5surf I implemented this feature :-)

[u5surf]

@wpjunior
Is there any conflict with this feature?
Of course, you can claim the your feature doesn't have to count unnecessary status code, but also we can consider that is not to introduce the redundant complexity of the code any more.

https://github.com/vozlt/nginx-module-vts?tab=readme-ov-file#to-calculate-traffic-for-detailed-http-status-code

BTW, I have an another suggestion for you about this pull request.
Can I contact with you via LinkedIn later?

[wpjunior]

Hi @u5surf,

Creating filters is a powerful tool. However, the current implementation seems to generate Prometheus metrics with two somewhat confusing dimensions: filter and filter_name. Making the status code a first-class attribute would simplify the metrics, making them easier for entry-level users to adopt.

Yeah, let's talk on LinkedIn!

[u5surf]

@wpjunior In addition some nit picks. At last I would be merge this if you will have finished some fixes. Thanks for a while!

@SuperQ Could you review this PR? Because this is a significant change for this module. For we have to make a decision to merge this, I'd like hear more collaboraters agree with the feature or not.

[u5surf]

@wpjunior Sorry for long time no response! I’ve decided to merge your PR because other reviewers had no response, I’ve no claimed you the change request anymore. Finally, could you fix a several chore conflict?
Thanks a lot and I hope to consider continuously the shape of the future about this module together.

[wpjunior]

Hi @u5surf, I am happy to made this git rebase,

done my friend

[u5surf]

@wpjunior Thanks! It’s all done!

[mur-me]

Huge thanks for this update @wpjunior

Already tried, works like a charm for the Prometheus metrics format 🙏