mirror of
https://github.com/librenms/librenms.git
synced 2024-10-07 16:52:45 +00:00
817cf42896
* Fixed device SNMP edit form (and better feedback) * snmp.inc.php: Fix SNMP Edit form (better feedback) * The feedback for Max Repeaters and Max OIDs is much better. The constant false errors on save have been corrected. The are no longer applicable and muted when SNMP is turned off. Their feedback now clearly shows the user what has been done. * Rename & relabel $no_checks as $force_save (Force Save) because that's simpler and more clearly defines what the code is doing when you turn it on. * Reorder the Force Save checkbox to be right above the Save button so it's even more clear what it does. Force Save and Save are the only elements on the form that do *not* have a database setting. They're ephemeral and now grouped as such. * Embedded comments about the use of === false as tests for setAttrib() and forgetAttrib() * snmp.inc.php: code climate updates, round 1 * remove blank lines * refactor duplicate code blocks * remove join on string * move print_messages to the bottom of the form * snmp.inc.php: code climate, round 2 * avoid deeply nested control flow statements (8) * snmp.inc.php: when to $force_save and $device_isssnmable * move $force_save outside of if ($snmp_enabled) so it works properly when snmp is disabled. * only check isSNMPable() if snmp is enabled * snmp.inc.php: move $device_snmp_details * $device_snmp_details are only needed for isSNMPable(), so only call them within the same contingency * snmp.inc.php: cleanup & hardening * add SNMP Settings header (to better mimic Device Settings, etc) * use Toastr for feedback, rather than print_message & print_error * after editing, always unset($_POST) and the other variables used for editing * unless the save is forced, if !$device_issnpable then don't save *anything* in the database and revert *all* previous form settings (some already were being reverted & some weren't) * more variables were ordered alphabetically, unused variables were and removed * for snmpver, don't use .toggle(); use .hide() & .show() so the form reverts properly when it fails * snmp.inc.php: clarify placeholder for max repeaters * snmp.inc.php: code climate, round 3 * correct blank line found at the end of control structure (2) * correct space before opening parenthesis of a function call prohibited (3)
505 lines
19 KiB
PHP
505 lines
19 KiB
PHP
<?php
|
|
|
|
use LibreNMS\Config;
|
|
|
|
if ($_POST['editing']) {
|
|
if (Auth::user()->hasGlobalAdmin()) {
|
|
$force_save = ($_POST['force_save'] == 'on');
|
|
$poller_group = isset($_POST['poller_group']) ? clean($_POST['poller_group']) : 0;
|
|
$snmp_enabled = ($_POST['snmp'] == 'on');
|
|
|
|
if ($snmp_enabled) {
|
|
$max_repeaters = clean($_POST['max_repeaters']);
|
|
$max_oid = clean($_POST['max_oid']);
|
|
$port = $_POST['port'] ? clean($_POST['port']) : Config::get('snmp.port');
|
|
$port_assoc_mode = clean($_POST['port_assoc_mode']);
|
|
$retries = clean($_POST['retries']);
|
|
$snmpver = clean($_POST['snmpver']);
|
|
$transport = $_POST['transport'] ? clean($_POST['transport']) : $transport = 'udp';
|
|
$timeout = clean($_POST['timeout']);
|
|
|
|
$update = array(
|
|
'poller_group' => $poller_group,
|
|
'port' => $port,
|
|
'port_association_mode' => $port_assoc_mode,
|
|
'snmp_disable' => 0,
|
|
'snmpver' => $snmpver,
|
|
'transport' => $transport,
|
|
);
|
|
|
|
if ($retries) {
|
|
$update['retries'] = $retries;
|
|
} else {
|
|
$update['retries'] = array('NULL');
|
|
}
|
|
|
|
if ($snmpver != "v3") {
|
|
$community = clean($_POST['community']);
|
|
$update['community' ] = $community;
|
|
}
|
|
|
|
if ($timeout) {
|
|
$update['timeout'] = $timeout;
|
|
} else {
|
|
$update['timeout'] = array('NULL');
|
|
}
|
|
|
|
$v3 = array();
|
|
if ($snmpver == "v3") {
|
|
$community = ''; // if v3 works, we don't need a community
|
|
|
|
$v3['authalgo'] = clean($_POST['authalgo']);
|
|
$v3['authlevel'] = clean($_POST['authlevel']);
|
|
$v3['authname'] = clean($_POST['authname']);
|
|
$v3['authpass'] = clean($_POST['authpass']);
|
|
$v3['cryptoalgo'] = clean($_POST['cryptoalgo']);
|
|
$v3['cryptopass'] = clean($_POST['cryptopass']);
|
|
|
|
$update = array_merge($update, $v3);
|
|
}
|
|
} else {
|
|
// snmp is disabled
|
|
$update['features'] = null;
|
|
$update['hardware'] = clean($_POST['hardware']);
|
|
$update['icon'] = null;
|
|
$update['os'] = $_POST['os'] ? clean($_POST['os_id']) : "ping";
|
|
$update['poller_group'] = $poller_group;
|
|
$update['snmp_disable'] = 1;
|
|
$update['sysName'] = $_POST['sysName'] ? clean($_POST['sysName']) : null;
|
|
$update['version'] = null;
|
|
}
|
|
|
|
$device_is_snmpable = false;
|
|
$rows_updated=0;
|
|
|
|
if ($force_save !== true && $snmp_enabled) {
|
|
$device_snmp_details = deviceArray($device['hostname'], $community, $snmpver, $port, $transport, $v3, $port_assoc_mode);
|
|
$device_issnmpable= isSNMPable($device_snmp_details);
|
|
}
|
|
|
|
if ($force_save === true || !$snmp_enabled || $device_issnmpable) {
|
|
// update devices table
|
|
$rows_updated = dbUpdate($update, 'devices', '`device_id` = ?', array($device['device_id']));
|
|
}
|
|
|
|
if ($snmp_enabled && ($force_save === true || $device_issnmpable)) {
|
|
// update devices_attribs table
|
|
|
|
// note:
|
|
// set_dev_attrib and del_dev_attrib *only* return (bool)
|
|
// setAttrib() returns true if it was set and false if it was not (e.g. it didn't change)
|
|
// forgetAttrib() returns true if it was deleted and false if it was not (e.g. it didn't exist)
|
|
// Symfony throws FatalThrowableError on error
|
|
|
|
$devices_attribs=array('snmp_max_repeaters', 'snmp_max_oid');
|
|
|
|
foreach ($devices_attribs as $devices_attrib) {
|
|
// defaults
|
|
$feedback_prefix=$devices_attrib;
|
|
$form_value=null;
|
|
$form_value_is_numeric=false; // does not need to be a number greater than zero
|
|
|
|
if ($devices_attrib == 'snmp_max_repeaters') {
|
|
$feedback_prefix="SNMP Max Repeaters";
|
|
$form_value=$max_repeaters;
|
|
$form_value_is_numeric=true;
|
|
}
|
|
|
|
if ($devices_attrib == 'snmp_max_oid') {
|
|
$feedback_prefix="SNMP Max OID";
|
|
$form_value=$max_oid;
|
|
$form_value_is_numeric=true;
|
|
}
|
|
|
|
$get_devices_attrib = get_dev_attrib($device, $devices_attrib);
|
|
$set_devices_attrib = false; // testing $set_devices_attrib === false is not a true indicator of a failure
|
|
|
|
if ($form_value != $get_devices_attrib && $form_value_is_numeric && is_numeric($form_value) && $form_value != 0) {
|
|
$set_devices_attrib=set_dev_attrib($device, $devices_attrib, $form_value);
|
|
}
|
|
|
|
if ($form_value != $get_devices_attrib && !$form_value_is_numeric) {
|
|
$set_devices_attrib=set_dev_attrib($device, $devices_attrib, $form_value);
|
|
}
|
|
|
|
if ($form_value != $get_devices_attrib && $form_value_is_numeric && !is_numeric($form_value)) {
|
|
$set_devices_attrib=del_dev_attrib($device, $devices_attrib);
|
|
}
|
|
|
|
if ($form_value != $get_devices_attrib && !$form_value_is_numeric && $form_value == '') {
|
|
$set_devices_attrib=del_dev_attrib($device, $devices_attrib);
|
|
}
|
|
|
|
if ($form_value != $get_devices_attrib && $set_devices_attrib) {
|
|
$set_devices_attrib = get_dev_attrib($device, $devices_attrib); // re-check the db value
|
|
}
|
|
|
|
if ($form_value != $get_devices_attrib && $form_value == $set_devices_attrib && (is_null($set_devices_attrib) || $set_devices_attrib == '')) {
|
|
$update_message[] = "$feedback_prefix deleted";
|
|
}
|
|
|
|
if ($form_value != $get_devices_attrib && $form_value == $set_devices_attrib && (!is_null($set_devices_attrib) && $set_devices_attrib != '')) {
|
|
$update_message[] = "$feedback_prefix updated to $set_devices_attrib";
|
|
}
|
|
|
|
if ($form_value != $get_devices_attrib && $form_value != $set_devices_attrib) {
|
|
$update_failed_message[] = "$feedback_prefix update failed.";
|
|
}
|
|
|
|
unset($get_devices_attrib, $set_devices_attrib);
|
|
}
|
|
unset($devices_attrib);
|
|
}
|
|
|
|
if ($rows_updated > 0) {
|
|
$update_message[] = 'Device record updated';
|
|
}
|
|
|
|
if ($snmp_enabled && ($force_save !== true && !$device_issnmpable)) {
|
|
$update_failed_message[] = "Could not connect to " . $device['hostname'] . " with those SNMP settings. To save anyway, turn on Force Save.";
|
|
$update_message[] = 'SNMP settings reverted';
|
|
}
|
|
|
|
if ($rows_updated == 0 && !isset($update_message) && !isset($update_failed_message)) {
|
|
$update_message[] = 'SNMP settings did not change';
|
|
}
|
|
}//end if (Auth::user()->hasGlobalAdmin())
|
|
}//end if ($_POST['editing'])
|
|
|
|
// the following unsets are for security; the database is authoritative
|
|
// i.e. prevent unintentional artifacts from being saved or used (again), posting around the form, etc.
|
|
unset($_POST);
|
|
// these are only used for editing and should not be used as-is
|
|
unset($force_save, $poller_group, $snmp_enabled);
|
|
unset($community, $max_repeaters, $max_oid, $port, $port_assoc_mode, $retries, $snmpver, $transport, $timeout);
|
|
|
|
// get up-to-date database values for use on the form
|
|
$device = dbFetchRow('SELECT * FROM `devices` WHERE `device_id` = ?', array($device['device_id']));
|
|
$max_oid = get_dev_attrib($device, 'snmp_max_oid');
|
|
$max_repeaters = get_dev_attrib($device, 'snmp_max_repeaters');
|
|
|
|
echo "<h3> SNMP Settings </h3>";
|
|
|
|
// use Toastr to print normal (success) messages, similar to Device Settings
|
|
if (isset($update_message)) {
|
|
$toastr_options=array();
|
|
|
|
if (is_array($update_message)) {
|
|
foreach ($update_message as $message) {
|
|
Toastr::success($message, null, $toastr_options);
|
|
}
|
|
}
|
|
|
|
if (is_string($update_message)) {
|
|
Toastr::success($update_message, null, $toastr_options);
|
|
}
|
|
|
|
unset($message, $toastr_options, $update_message);
|
|
}
|
|
|
|
// use Toastr:error to call attention to the problem; don't let it time out
|
|
if (isset($update_failed_message)) {
|
|
$toastr_options=array();
|
|
$toastr_options["closeButton"]=true;
|
|
$toastr_options["extendedTimeOut"]=0;
|
|
$toastr_options["timeOut"]=0;
|
|
|
|
if (is_array($update_failed_message)) {
|
|
foreach ($update_failed_message as $error) {
|
|
Toastr::error($error, null, $toastr_options);
|
|
}
|
|
}
|
|
|
|
if (is_string($update_failed_message)) {
|
|
Toastr::error($update_failed_message, null, $toastr_options);
|
|
}
|
|
|
|
unset($error, $update_failed_message);
|
|
}
|
|
|
|
echo "
|
|
<form id='edit' name='edit' method='post' action='' role='form' class='form-horizontal'>
|
|
" . csrf_field() . "
|
|
<div class='form-group'>
|
|
<label for='hardware' class='col-sm-2 control-label'>SNMP</label>
|
|
<div class='col-sm-4'>
|
|
<input type='checkbox' id='snmp' name='snmp' data-size='small' onChange='disableSnmp(this);'".($device['snmp_disable'] ? "" : " checked").">
|
|
</div>
|
|
</div>
|
|
<div id='snmp_override' style='display: ".($device['snmp_disable'] ? "block" : "none").";'>
|
|
<div class='form-group'>
|
|
<label for='sysName' class='col-sm-2 control-label'>sysName (optional)</label>
|
|
<div class='col-sm-4'>
|
|
<input id='sysName' class='form-control' name='sysName' value='".$device['sysName']."'/>
|
|
</div>
|
|
</div>
|
|
<div class='form-group'>
|
|
<label for='hardware' class='col-sm-2 control-label'>Hardware (optional)</label>
|
|
<div class='col-sm-4'>
|
|
<input id='hardware' class='form-control' name='hardware' value='".$device['hardware']."'/>
|
|
</div>
|
|
</div>
|
|
<div class='form-group'>
|
|
<label for='os' class='col-sm-2 control-label'>OS (optional)</label>
|
|
<div class='col-sm-4'>
|
|
<input id='os' class='form-control' name='os' value='" . Config::get("os.{$device['os']}.text") . "'/>
|
|
<input type='hidden' id='os_id' class='form-control' name='os_id' value='".$device['os']."'/>
|
|
</div>
|
|
</div>
|
|
</div>
|
|
<div id='snmp_conf' style='display: ".($device['snmp_disable'] ? "none" : "block").";'>
|
|
<input type=hidden name='editing' value='yes'>
|
|
<div class='form-group'>
|
|
<label for='snmpver' class='col-sm-2 control-label'>SNMP Details</label>
|
|
<div class='col-sm-1'>
|
|
<select id='snmpver' name='snmpver' class='form-control input-sm' onChange='changeForm();'>
|
|
<option value='v1'>v1</option>
|
|
<option value='v2c' ".($device['snmpver'] == 'v2c' ? 'selected' : '').">v2c</option>
|
|
<option value='v3' ".($device['snmpver'] == 'v3' ? 'selected' : '').">v3</option>
|
|
</select>
|
|
</div>
|
|
<div class='col-sm-2'>
|
|
<input type='text' name='port' placeholder='port' class='form-control input-sm' value='" . ($device['port'] == Config::get('snmp.port') ? "" : $device['port']) . "'>
|
|
</div>
|
|
<div class='col-sm-1'>
|
|
<select name='transport' id='transport' class='form-control input-sm'>";
|
|
foreach (Config::get('snmp.transports') as $transport) {
|
|
echo "<option value='".$transport."'";
|
|
if ($transport == $device['transport']) {
|
|
echo " selected='selected'";
|
|
}
|
|
|
|
echo '>'.$transport.'</option>';
|
|
}
|
|
|
|
echo " </select>
|
|
</div>
|
|
</div>
|
|
<div class='form-group'>
|
|
<div class='col-sm-2'>
|
|
</div>
|
|
<div class='col-sm-1'>
|
|
<input id='timeout' name='timeout' class='form-control input-sm' value='".($device['timeout'] ? $device['timeout'] : '')."' placeholder='seconds' />
|
|
</div>
|
|
<div class='col-sm-1'>
|
|
<input id='retries' name='retries' class='form-control input-sm' value='".($device['timeout'] ? $device['retries'] : '')."' placeholder='retries' />
|
|
</div>
|
|
</div>
|
|
<div class='form-group'>
|
|
<label for='port_assoc_mode' class='col-sm-2 control-label'>Port Association Mode</label>
|
|
<div class='col-sm-1'>
|
|
<select name='port_assoc_mode' id='port_assoc_mode' class='form-control input-sm'>
|
|
";
|
|
|
|
foreach (get_port_assoc_modes() as $pam_id => $pam) {
|
|
echo " <option value='$pam_id'";
|
|
|
|
if ($pam_id == $device['port_association_mode']) {
|
|
echo " selected='selected'";
|
|
}
|
|
|
|
echo ">$pam</option>\n";
|
|
}
|
|
|
|
echo " </select>
|
|
</div>
|
|
</div>
|
|
<div class='form-group'>
|
|
<label for='max_repeaters' class='col-sm-2 control-label'>Max Repeaters</label>
|
|
<div class='col-sm-1'>
|
|
<input id='max_repeaters' name='max_repeaters' class='form-control input-sm' value='".$max_repeaters."' placeholder='max repeaters' />
|
|
</div>
|
|
</div>
|
|
<div class='form-group'>
|
|
<label for='max_oid' class='col-sm-2 control-label'>Max OIDs</label>
|
|
<div class='col-sm-1'>
|
|
<input id='max_oid' name='max_oid' class='form-control input-sm' value='".$max_oid."' placeholder='max oids' />
|
|
</div>
|
|
</div>
|
|
<div id='snmpv1_2'>
|
|
<div class='form-group'>
|
|
<label class='col-sm-3 control-label text-left'><h4><strong>SNMPv1/v2c Configuration</strong></h4></label>
|
|
</div>
|
|
<div class='form-group'>
|
|
<label for='community' class='col-sm-2 control-label'>SNMP Community</label>
|
|
<div class='col-sm-4'>
|
|
<input id='community' class='form-control' name='community' value='".$device['community']."'/>
|
|
</div>
|
|
</div>
|
|
</div>
|
|
<div id='snmpv3'>
|
|
<div class='form-group'>
|
|
<label class='col-sm-3 control-label'><h4><strong>SNMPv3 Configuration</strong></h4></label>
|
|
</div>
|
|
<div class='form-group'>
|
|
<label for='authlevel' class='col-sm-2 control-label'>Auth Level</label>
|
|
<div class='col-sm-4'>
|
|
<select id='authlevel' name='authlevel' class='form-control'>
|
|
<option value='noAuthNoPriv'>noAuthNoPriv</option>
|
|
<option value='authNoPriv' ".($device['authlevel'] == 'authNoPriv' ? 'selected' : '').">authNoPriv</option>
|
|
<option value='authPriv' ".($device['authlevel'] == 'authPriv' ? 'selected' : '').">authPriv</option>
|
|
</select>
|
|
</div>
|
|
</div>
|
|
<div class='form-group'>
|
|
<label for='authname' class='col-sm-2 control-label'>Auth User Name</label>
|
|
<div class='col-sm-4'>
|
|
<input type='text' id='authname' name='authname' class='form-control' value='".$device['authname']."' autocomplete='off'>
|
|
</div>
|
|
</div>
|
|
<div class='form-group'>
|
|
<label for='authpass' class='col-sm-2 control-label'>Auth Password</label>
|
|
<div class='col-sm-4'>
|
|
<input type='password' id='authpass' name='authpass' class='form-control' value='".$device['authpass']."' autocomplete='off'>
|
|
</div>
|
|
</div>
|
|
<div class='form-group'>
|
|
<label for='authalgo' class='col-sm-2 control-label'>Auth Algorithm</label>
|
|
<div class='col-sm-4'>
|
|
<select id='authalgo' name='authalgo' class='form-control'>
|
|
<option value='MD5'>MD5</option>
|
|
<option value='SHA' ".($device['authalgo'] === 'SHA' ? 'selected' : '').">SHA</option>
|
|
</select>
|
|
</div>
|
|
</div>
|
|
<div class='form-group'>
|
|
<label for='cryptopass' class='col-sm-2 control-label'>Crypto Password</label>
|
|
<div class='col-sm-4'>
|
|
<input type='password' id='cryptopass' name='cryptopass' class='form-control' value='".$device['cryptopass']."' autocomplete='off'>
|
|
</div>
|
|
</div>
|
|
<div class='form-group'>
|
|
<label for='cryptoalgo' class='col-sm-2 control-label'>Crypto Algorithm</label>
|
|
<div class='col-sm-4'>
|
|
<select id='cryptoalgo' name='cryptoalgo' class='form-control'>
|
|
<option value='AES'>AES</option>
|
|
<option value='DES' ".($device['cryptoalgo'] === 'DES' ? 'selected' : '').">DES</option>
|
|
</select>
|
|
</div>
|
|
</div>
|
|
</div>";
|
|
|
|
?>
|
|
|
|
</div>
|
|
<?php
|
|
|
|
if (Config::get('distributed_poller') === true) {
|
|
echo '
|
|
<div class="form-group">
|
|
<label for="poller_group" class="col-sm-2 control-label">Poller Group</label>
|
|
<div class="col-sm-4">
|
|
<select name="poller_group" id="poller_group" class="form-control input-sm">
|
|
<option value="0"> Default poller group</option>
|
|
';
|
|
|
|
foreach (dbFetchRows('SELECT `id`,`group_name` FROM `poller_groups`') as $group) {
|
|
echo '<option value="'.$group['id'].'"';
|
|
if ($device['poller_group'] == $group['id']) {
|
|
echo ' selected';
|
|
}
|
|
|
|
echo '>'.$group['group_name'].'</option>';
|
|
}
|
|
|
|
echo '
|
|
</select>
|
|
</div>
|
|
</div>
|
|
';
|
|
}//end if
|
|
?>
|
|
|
|
<div class="form-group">
|
|
<label for="force_save" class="control-label col-sm-2">Force Save</label>
|
|
<div class="col-sm-9">
|
|
<input type="checkbox" name="force_save" id="force_save" data-size="small">
|
|
</div>
|
|
</div>
|
|
|
|
<div class="row">
|
|
<div class="col-md-1 col-md-offset-2">
|
|
<button type="submit" name="Submit" class="btn btn-success"><i class="fa fa-check"></i> Save</button>
|
|
</div>
|
|
</div>
|
|
</form>
|
|
|
|
<script>
|
|
$('[name="force_save"]').bootstrapSwitch();
|
|
|
|
function changeForm() {
|
|
snmpVersion = $("#snmpver").val();
|
|
if(snmpVersion == 'v1' || snmpVersion == 'v2c') {
|
|
$('#snmpv1_2').show();
|
|
$('#snmpv3').hide();
|
|
}
|
|
else if(snmpVersion == 'v3') {
|
|
$('#snmpv1_2').hide();
|
|
$('#snmpv3').show();
|
|
}
|
|
}
|
|
function disableSnmp(e) {
|
|
if(e.checked) {
|
|
$('#snmp_conf').show();
|
|
$('#snmp_override').hide();
|
|
} else {
|
|
$('#snmp_conf').hide();
|
|
$('#snmp_override').show();
|
|
}
|
|
}
|
|
|
|
var os_suggestions = new Bloodhound({
|
|
datumTokenizer: Bloodhound.tokenizers.obj.whitespace('text'),
|
|
queryTokenizer: Bloodhound.tokenizers.whitespace,
|
|
remote: {
|
|
url: "ajax_ossuggest.php?term=%QUERY",
|
|
filter: function (output) {
|
|
return $.map(output, function (item) {
|
|
return {
|
|
text: item.text,
|
|
os: item.os,
|
|
};
|
|
});
|
|
},
|
|
wildcard: "%QUERY"
|
|
}
|
|
});
|
|
os_suggestions.initialize();
|
|
$('#os').typeahead({
|
|
hint: true,
|
|
highlight: true,
|
|
minLength: 1,
|
|
classNames: {
|
|
menu: 'typeahead-left'
|
|
}
|
|
},
|
|
{
|
|
source: os_suggestions.ttAdapter(),
|
|
async: true,
|
|
displayKey: 'text',
|
|
valueKey: 'os',
|
|
templates: {
|
|
suggestion: Handlebars.compile('<p> {{text}}</p>')
|
|
},
|
|
limit: 20
|
|
});
|
|
|
|
$("#os").on("typeahead:selected typeahead:autocompleted", function(e,datum) {
|
|
$("#os_id").val(datum.os);
|
|
$("#os").html('<mark>' + datum.text + '</mark>');
|
|
});
|
|
|
|
$("[name='snmp']").bootstrapSwitch('offColor','danger');
|
|
|
|
<?php
|
|
if ($device['snmpver'] == 'v3') {
|
|
echo "$('#snmpv1_2').hide();";
|
|
echo "$('#snmpv3').show();";
|
|
} else {
|
|
echo "$('#snmpv1_2').show();";
|
|
echo "$('#snmpv3').hide();";
|
|
}
|
|
|
|
?>
|
|
</script>
|