Project

General

Profile

Actions
Copy link

Bug #4880

closed

ntopng - multiple issues (needs a major package rewrite)

Added by Kill Bill almost 9 years ago. Updated over 8 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
ntop
Target version:
-
Start date:
07/25/2015
Due date:
% Done:

0%

Estimated time:
Plus Target Version:
Affected Version:
All
Affected Plus Version:
Affected Architecture:
All

Description

1/ All settings lost on reinstall/upgrade. After reinstall, I'm left with this in config.xml

<ntopng>
        <config>
                <__csrf_magic>sid:3c785b8e18dc3e5eed8beb64663550d794fd6ad6,1437813179</__csrf_magic>
                <id>ntopng</id>
                <mode>reinstallpkg</mode>
                <pkgconfirm>Confirm</pkgconfirm>
                <password>admin</password>
        </config>
</ntopng>

2/ Every second reinstall/upgrade fails (stops) at this point:

Custom commands...
Executing custom_php_global_functions()...

(which is mostly likely related to #3).

3/ All the functions stuff needs to go to separate inc file and only be called in places where it makes sense. When you (re)install the thing, basically exact same code is run no less than 3 times:

Custom commands...
Executing custom_php_global_functions()...done.
Executing custom_php_install_command()...done.
Executing custom_php_resync_config_command()...done.

Both custom_php_install_command() and custom_php_resync_config_command() call sync_package_ntopng() so it's run 3 times, which is totally pointless.

(Will try to produce a pull req if the time permits.)

Actions
Copy link

Also available in: Atom PDF