Skip to content
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

It's possible to add a lower position counter through CRON #78

Open
eduardomozart opened this issue Nov 20, 2020 · 0 comments
Open

It's possible to add a lower position counter through CRON #78

eduardomozart opened this issue Nov 20, 2020 · 0 comments

Comments

@eduardomozart
Copy link
Contributor

eduardomozart commented Nov 20, 2020

  • Version of the plugin : 1.7.0

  • Version of your GLPI : 9.5.2

  • Steps to reproduce (which actions have you made) :

The specification documentation has a diagram that shows the error management of the plug-in - https://forge.glpi-project.org/projects/glpi/wiki/SNMP_printer_counters#27-Gestion-des-erreurs

It states that, if the Counter < Last record, the Counter isn't added, but it's possible when adding it through the CRON script.

E.g. The following screenshot:

2020-11-20 19_30_00-Window

  1. This is an intentionaly added "Manual" record with a higher value than the last ones.
  2. The second is an attempt to add a "Manual" record with a lower value than the last one (it fails, as it should).
  3. The third is the result of the CRON script: it adds the record even if it's lower than the last one.
  • Expected result :

The third result should return an "Record error" instead of "Success".

  • Actual result :

The record is written into the database.

  • URL of the page :

<GLPI_ROOT>/plugins/printercounters/scripts/run.php

  • Screenshot of the problem (if pertinent) :
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant