New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

sometime agent don't send name #2358

Merged
merged 3 commits into from Dec 5, 2017

Conversation

Projects
None yet
4 participants
@orthagh
Contributor

orthagh commented Nov 23, 2017

500MB of logs in one night :D
here is an extract:

2017-11-23 09:23:49 [17134@wgpi1]
  *** PHP Notice(8): Undefined index: name
  Backtrace :
  ...nventory/inc/inventorycomputerlib.class.php:188
  ...ry/inc/inventorycomputerinventory.class.php:573 PluginFusioninventoryInventoryComputerLib->updateComputer()
  ...inventory/inc/inventoryruleimport.class.php:790 PluginFusioninventoryInventoryComputerInventory->rulepassed()
  inc/rule.class.php:1436                            PluginFusioninventoryInventoryRuleImport->executeActions()
  inc/rulecollection.class.php:1461                  Rule->process()
  ...ry/inc/inventorycomputerinventory.class.php:378 RuleCollection->processAllRules()
  ...ry/inc/inventorycomputerinventory.class.php:126 PluginFusioninventoryInventoryComputerInventory->sendCriteria()
  ...fusioninventory/inc/communication.class.php:234 PluginFusioninventoryInventoryComputerInventory->import()
  ...fusioninventory/inc/communication.class.php:463 PluginFusioninventoryCommunication->import()
  plugins/fusioninventory/front/communication.php:93 PluginFusioninventoryCommunication->handleOCSCommunication()
  plugins/fusioninventory/index.php:59               include_once()
@flegastelois

This comment has been minimized.

Contributor

flegastelois commented Nov 23, 2017

Maybe we can do the same things for "Monitor" (around line ~1518) and "Printer" (around line ~1604).

@wawax

wawax approved these changes Dec 5, 2017

@wawax wawax merged commit b3a6c35 into fusioninventory:glpi9.2 Dec 5, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@wawax wawax added the bug label Dec 5, 2017

mohierf added a commit to mohierf/fusioninventory-for-glpi that referenced this pull request Jan 10, 2018

sometime agent don't send name (fusioninventory#2358)
Fix errors in php log file when a computer, a monitor or a printer doesn't have a name

mohierf added a commit to mohierf/fusioninventory-for-glpi that referenced this pull request Jan 19, 2018

sometime agent don't send name (fusioninventory#2358)
Fix errors in php log file when a computer, a monitor or a printer doesn't have a name

@wawax wawax added this to the 9.2+2.0 milestone Jan 29, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment