Cacti / plugin_flowview

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

Create filter has select all and and all option leading to database column length issue #174

Closed bmfmancini closed 4 months ago

bmfmancini commented 4 months ago

when creating a filter the user has an option to check all which will insert each protocol number into the protocols column

or any which inserts a single value into the column Protocols is set to varchar(32) so when checked all is selected the column is not long enough I think if the user selects all check all should be disabled

image

2. row id: 2 name: test2 device_id: 0 template_id: -1 ex_addr: -1 timespan: 0 startdate: 2024-07-22 14:31 enddate: 2024-07-22 15:01 tosfields: tcpflags: protocols: 0, 6, 17, 1, 2, 3, 4, 5, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 18, 19, 20, 21, 22, 23, 24, 25, 26, 27, 28, 29, 30, 31, 32, 33, 34, 35, 36, 37, 38, 39, 40, 41, 42, 43, 44, 45, 46, 47, 48, 49, 50, 51, 58, 59, 60, 73, 81, 88, 89, 92, 94, 98 sourceip: sourceport: sourceinterface: sourceas: destip: destport: destinterface: destas: statistics: 10 printed: 0 includeif: 2 sortfield: 0 cutofflines: 20 cutoffoctets: 0 resolve: Y graph_type: bar graph_height: 300 panel_table: panel_bytes: panel_packets: panel_flows: 2 rows in set (0.000 sec)

MariaDB [flowview]> select max(length(protocols)) from plugin_flowview_queries \G 1. row max(length(protocols)): 236 1 row in set (0.000 sec)