ngxtop icon indicating copy to clipboard operation
ngxtop copied to clipboard

Support for 'include' directives added

Open takeshixx opened this issue 10 years ago • 4 comments

This update allows ngxtop to parse files that are included in the Nginx config via 'include' directives. Useful for large deployments where each vhost is located in a separate config file like:

include /usr/local/www/conf/*.conf;

takeshixx avatar Apr 01 '14 14:04 takeshixx

Thanks, support for include directive would be very helpful indeed. However I think you missed some complications with current implementation:

  • The included file can have other include directives
  • The include file can be specified as relative path to the main nginx config

lebinh avatar Apr 01 '14 16:04 lebinh

It was just a quick fix to make ngxtop run on my reverse proxy setup. I'll may fix those two issues in the next few days.

takeshixx avatar Apr 02 '14 09:04 takeshixx

+1

lostsnow avatar Dec 11 '15 09:12 lostsnow

Merged: https://github.com/ngxtop/ngxtop/commit/f09560351c469d9d40b128afaab10d0c5856256e

oxr463 avatar Apr 12 '20 01:04 oxr463