search

OCSInventory-NG / OCSInventory-ocsreports

Webconsole for OCS Inventory NG
https://www.ocsinventory-ng.org
GNU General Public License v2.0
229 stars 150 forks source link

[BUG/FEATURE] update OCS #1581

Closed Parnoud closed 5 months ago

Parnoud commented 8 months ago

OCS Inventory version Version : all with ldap/ldaps or ssologin

Describe the bug After a maj, user cannot login with ldap/laps if enable before because the file is not backup or saved followin the documentation.

To Reproduce Steps to reproduce the behavior: Update OCS console to newer version, copy old file with with new file. Restart the service and try to login with ldaps

Expected behavior After maj, user should login in console as usual

Solution in Wiki OCS add a point for /usr/share/ocsinventory-reports/ocsreports/var.php and auth type line 154.

Solution bis Add a feature in setup.sh to find value he need in old config file if existe for update. Sometime, the file /etc/apache2/conf-available/z-ocsinventory-server.conf /etc/apache2/conf-available/zz-ocsinventory-restapi.conf /etc/apache2/conf-available/ocsinventory-reports.conf /usr/share/ocsinventory-reports/ocsreports/dbconfig.inc.php arent the same between version. And maybe if this is implemented, add the auto remove of /usr/share/ocsinventory-reports/ocsreports/install.php

Lea9250 commented 7 months ago

Hi @Parnoud,

I agree that this should at least be mentioned within the documentation. I've made a PR for this : https://github.com/OCSInventory-NG/Wiki/pull/184

Thank you for bringing it to our attention. We'll also consider the changes you proposed as an alternative solution.

Best regards, Léa