search

Sean-Der / fail2web

a fail2ban GUI powered by fail2rest
536 stars 48 forks source link

Adding an option to deactivate pie chart #33

Open MaikJaek opened 7 years ago

MaikJaek commented 7 years ago

Dear @Sean-Der ,

could you please add a configuration option to not render the pie chart and only render the table of entries instead? With thousands of bans on a system, browsers may regularly crash trying to render the pie chart.

Thanks in advance! Maik

Sean-Der commented 7 years ago

Hey @MaikJaek

That sounds like a useful feature! I haven't had a chance to work on fail2web in a while, so I will take some time this weekend to update all the libraries etc.. and add that feature.

Will keep you updated/close this when it is done! thanks

MaikJaek commented 7 years ago

Hi @Sean-Der ,

Thanks. I appreciate the effort! I tried to fix it myself. However, the used libraries are so old that I cannot manage to build it (and I've tried with very old NodeJS versions).

I did a quick hack on bundle.js to resolve the immediate need for my company. Nevertheless, even rendering just 1000 banned ips in the list takes a long time. Maybe this can be fixed using angular 2 or a different way of creating and sorting the table.

Feel free to contact me, if you need any more input or help. I'll see what I can do.

Thanks in advance for taking care!

mbautista commented 4 years ago

Hello, Thanks you, fail2web is a great tool ! :) We also had this issue : the history chart makes the browser unresponsive with important number of IPs. I've commented the following lines in web/bundle.js around lines 109-110 to disable the history chart (as we only need the ip list) : 

        //$scope.$watch('global.bans', rebuildCharts, true);
        //$scope.$watch('activeJail.name', rebuildCharts, true);

Thank you :)