Cacti / plugin_gexport

Cacti Graph Export Plugin
GNU General Public License v2.0
12 stars 9 forks source link

CMDPHP PHP ERROR WARNING #57

Closed WirthmU closed 1 year ago

WirthmU commented 1 year ago

Describe the bug CMDPHP PHP ERROR WARNING in cacti log file complaining about

To Reproduce Steps to reproduce the behavior:

 28-02-2023 13:30:10 - CMDPHP PHP ERROR WARNING Backtrace: (/plugins/gexport/poller_export.php[99]:graph_export(), /plugins/gexport/functions.php[120]:run_export(), /plugins/gexport/functions.php[186]:export_ftp_php_execute(), /plugins/gexport/functions.php[1133]:export_ftp_php_uploaddir(), /plugins/gexport/functions.php[1191]:export_ftp_php_uploaddir(), /plugins/gexport/functions.php[1187]:ftp_mkdir(), CactiErrorHandler())
28-02-2023 13:30:10 - ERROR PHP WARNING in Plugin 'gexport': ftp_mkdir(): images: File exists. in file: /usr/share/cacti/site/plugins/gexport/functions.php on line: 1187 

Expected behavior no gexport error in cacti log

Screenshots If applicable, add screenshots to help explain your problem.

Plugin (please complete the following information):

Desktop (please complete the following information):

Smartphone (please complete the following information):

Additional context Add any other context about the problem here.

TheWitness commented 1 year ago

You need to enable the sensitization of the remote directory if you wish to ignore this error at the current time. It's not quite a bug per-se. It's a downside of using the FTP method.

TheWitness commented 1 year ago

image

TheWitness commented 1 year ago

I pushed an update to functions.php. Update it and before you enable sensitization, see if it fixes it.

WirthmU commented 1 year ago

The updated functions.php did the trick - no more warnings.

Thank you

TheWitness commented 1 year ago

Cool!