Cacti / plugin_flowview

FlowView Plugin for Cacti
GNU General Public License v2.0
18 stars 10 forks source link

Use Arin to Find Unregistered Domains wont disable #136

Closed bmfmancini closed 3 months ago

bmfmancini commented 3 months ago

Need to dig abit more but even when Use Arin to Find Unregistered Domains is disabled in settings flowview_get_owner_from_arin still gets called

Settings table is updated as expected

TheWitness commented 3 months ago

This is due to the setting being cached once the flow_collector.php is started. Maybe we need to send a hup to the service to tell it to reload this key setting. What do you think?

TheWitness commented 3 months ago

Key settings, I guess the DNS IP's under Mail/Reporting/DNS would also be key....

image

bmfmancini commented 3 months ago

Yeah either a hup Or a shell exec to systemd to restart the daemon

On Sat, Jun 15, 2024, 21:58 TheWitness @.***> wrote:

Key settings, I guess the DNS IP's under Mail/Reporting/DNS would also be key....

image.png (view on web) https://github.com/Cacti/plugin_flowview/assets/1439914/fe746ebf-7ef2-4762-b19e-c08c28874a22

— Reply to this email directly, view it on GitHub https://github.com/Cacti/plugin_flowview/issues/136#issuecomment-2171010494, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADGEXTHZTNBDGCC55KWAMDDZHTWMDAVCNFSM6AAAAABJLOBS62VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZRGAYTANBZGQ . You are receiving this because you authored the thread.Message ID: @.***>

TheWitness commented 3 months ago

This should be resolved now under these two commits:

429e1072a80d040cead60e0c244afb2f1b6d406c and 569e408a304a37336798412147987a4bfa4ddd8a