miken32
Repos
18
Followers
7

Events

issue comment
[nagios server]/nagvis/frontend/nagvis-js/index.php drop error 500 after login

The error PHP Fatal error: Class 'PDO' not found is quite clear. Install PDO support on your server.

Created at 5 days ago
Add Icinga DB support

(in case there is a JSON error you'll only get an undefined variable error, and not see the JSON error)

Is the error triggered somewhere in this module? And does the error get displayed correctly with the ndomy backend?

No it was a deprecation error in Nagvis (NagVis/nagvis#335) after updating to PHP 8.1. No idea if it works otherwise, we only have icingadb for storage, but it's Nagvis displaying the error so I imagine it's the same either way?

Created at 1 week ago
Add Icinga DB support

Other than my nitpick above (in case there is a JSON error you'll only get an undefined variable error, and not see the JSON error) I tested latest branch and it works great for us.

Created at 1 week ago
opened issue
Implicit conversions trigger deprecation warnings in PHP 8.1

In share/server/core/classes/objects/NagVisStatefulObject.php at lines 265 and 279 a string is implicitly converted to an integer; this needs to be explicitly converted to avoid deprecation warnings.

I was going to submit a PR to resolve this issue; is PHP 5.0 compatibility seriously a requirement? I could almost forgive 5.6, but we're even 5 years out from that being EOL.

Created at 1 week ago

add a bit of documentation

Created at 1 week ago
Created at 1 week ago

add smartzone ap count checkerg

Created at 1 week ago

align arguments with manubulon, use bash arrays properly

align options with manubulon, move snmp into function, use (()) math

new doctl output, check droplet count against account limit

update to support snmpv3

Created at 1 week ago

fix some access checks that became too strict after commit 5864d47 (or in some cases commit d9270fe)

Created at 1 week ago

fix some access checks that became too strict after commit 5864d47 (or in some cases commit d9270fe)

Created at 1 week ago

make permission bits explicit, move rights check to helper function

move constants to a class; more stuff to follow

move admin access check into a function remove some redundant checks

get rid of more global variables

Created at 1 week ago
Created at 1 week ago
check_snmp can conflict with settings in snmp.conf

Probably also worth mentioning, the output read loop in main() breaks early due to lack of separator character ":" as excpected. But the check at the end for calculate_rate doesn't take the possibility of an early exit from the loop into account. It should also be checking if total_oids > 0. We were seeing verbose outputs like this, but it probably should have been an error condition:

Previous State for 0=`????
/usr/bin/snmpget -Le -t 3 -r 5 -m ALL -v 2c [context] [authpriv] example.com:161 ifInOctets.24
IF-MIB::ifInOctets.24 4103007855
State string=`?b??
SNMP RATE UNKNOWN - | 
Created at 2 weeks ago
check_snmp can conflict with settings in snmp.conf

In particular, we were trying to use this tool for rate checking; the checks always came back as "UNKNOWN." We enabled verbose mode and went through check_snmp.c until we realized that our system-wide configuration setting of quickPrinting true was messing with the results.

If this tool is expecting a certain output it should be asking for it with the appropriate options (in this case by explicitly passing --quickPrinting=false to snmpget) or – more properly – importing the net-snmp libraries and doing the checks in C instead of running CLI utilities.

Created at 2 weeks ago

don't show login if already authenticated

Created at 2 weeks ago

fix filename

Created at 3 weeks ago

continue pissing into the wind that is Icinga's hositility to open source

Created at 3 weeks ago

fix spec error

update versions

Created at 3 weeks ago
issue comment
Packaging Adjustments

@lbetz not to mention anyone that uses seliux. sure this software is open source, but guess you'll have to pay if you want to make it useful.

Created at 3 weeks ago
issue comment
php8 modernizing
  • From install.php, both get_magic_quotes_gpc() and MB_OVERLOAD_STRING don't exist in modern PHP versions.
  • From application/helpers/rrd.php both alerter() and alerter_gr() are missing a default value for $unit

Supporting PHP 5.6 seems like a bit of a lost cause. If they want to run legacy systems, let them use legacy code.

Created at 3 weeks ago

fix height on QR code labels

Created at 1 month ago

include the authy icon in the description for additional context

check for invalid filenames instead of crashing

Created at 1 month ago

add print styles for making physical backups

Created at 1 month ago
pull request opened
Export improvements

Dunno if this is still maintained, but I made use of it today so thought I'd contribute back.

The main goal was some more functional HTML output. Blurring out the QR codes makes it easier to take a picture of a single code on the screen, and putting them in a grid makes it easier to find the ones you're looking for without lots of scrolling.

While I was adding that, I noticed the code was a bit of a mess so I cleaned it up some. Whitespace is now consistent, files are not getting opened and closed inside a loop, and there's less repetition.

This worked nicely for me to grab my tokens, thanks!

Created at 1 month ago

fix some errors (I've never used go before)

Created at 1 month ago

clean up whitespace, add prettier HTML

Created at 1 month ago
Created at 1 month ago

avoid some notices in the logs

Created at 1 month ago