Bug 20945 - изменить текст в readme.alt - пароль менять...
Summary: изменить текст в readme.alt - пароль менять...
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: ocsinventory-reports (show other bugs)
Version: unstable
Hardware: all Linux
: P3 normal
Assignee: zidex@altlinux.org
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-08-04 10:20 MSD by Mike Lykov
Modified: 2009-11-26 21:47 MSK (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Mike Lykov 2009-08-04 10:20:56 MSD
в readme.alt есть такой текст:

1."  * GRANT ALL PRIVILEGES ON *.* TO ocs@localhost IDENTIFIED BY 'ocs';"

 по сути, это означает "дать этому пользователю, пароль которого совпадает с его именем, права НА ВСЁ ВЕЗДЕ".
мне видится это несколько небезопасным, в то время как достаточно
 GRANT ALL PRIVILEGES ON ocsweb.* TO ocs@localhost IDENTIFIED BY 'ocs';

2. "Имя базы (ocsweb), пользователь (ocs) и возможно пароль (ocs) в коде
прибито гвоздями, не рекомендуется изменять эти значения во избежание
недоразумений."

в конфиге apache это "PerlSetVar OCS_DB_PWD <пароль>" в 
/etc/httpd2/conf/extra-available/ocsinventory.conf

можно исправить руками для comm.server.
для report надо исправить в 1 месте для использования в install.php
--- install-ocs.php     2009-01-12 03:06:29 +0400
+++ install.php 2009-08-04 11:09:54 +0500

@@ -161,7 +161,7 @@
 }

 if($_POST["fin"]=="fin") {
-       if(!@mysql_connect($_POST["host"],"ocs","ocs")) {
+       if(!@mysql_connect($_POST["host"],"ocs","пароль")) {

и в одном месте для того чтобы писать конфиг (потом используемый)
@@ -183,7 +183,7 @@
        die();
 }

-fwrite($ch,"<?php \n\$_SESSION[\"SERVEUR_SQL\"]=\"".$_POST["host"]."\";\n\$_SES
SION[\"COMPTE_BASE\"]=\"ocs\";\n\$_SESSION[\"PSWD_BASE\"]=\"ocs\";\n?>");
+fwrite($ch,"<?php \n\$_SESSION[\"SERVEUR_SQL\"]=\"".$_POST["host"]."\";\n\$_SES
SION[\"COMPTE_BASE\"]=\"ocs\";\n\$_SESSION[\"PSWD_BASE\"]=\"пароль\";\n?>");
 fclose($ch);


вот если бы вместо "пароль" сюда вставить переменную, которая содержит пароль пользователя, _уже получаемый_ на первом шаге..
ибо слишком уж это тупо у них(ocs-ng) сделано ;)
Comment 1 Mike Lykov 2009-11-06 14:08:36 MSK
протестировал 
ocsinventory-reports-1.02.1-alt1

в описанном тут плане с виду все ок (текст readme, работа инсталлятора)
Comment 2 Mike Lykov 2009-11-06 14:39:21 MSK
установил в соотв. как написано в readme, но подцепил к существующей базе.
(использовал инсталлятор, он распознал что база уже существует и не стал ее удалять/создавать/выдавать ошибки)

в первый заход вместо русского было нечто невернокодированное, во второй - русский  на месте (странно ;) 

в интерфейсе - есть ("Последнее обновление:	11/06/2009 14:34:06")
в настройках конкр. машины - есть ("Режим работы СборщикаIP ..") (в моем варианте , установленном с сайта тут английский, неполный перевод)
в деталях конфигурации  - есть ("Физическая память  	 A0 (No ECC) ")

насчет "обновления со старой версии" не проверял, ocsinventory-agent тоже не проверял.
Comment 3 Pavel Zilke 2009-11-06 19:16:16 MSK
не понял вот это:(В ответ на комментарий №2)

> в интерфейсе - есть ("Последнее обновление:    11/06/2009 14:34:06")
> в настройках конкр. машины - есть ("Режим работы СборщикаIP ..") (в моем
> варианте , установленном с сайта тут английский, неполный перевод)
> в деталях конфигурации  - есть ("Физическая память       A0 (No ECC) ")

чего и на что нужно исправить?
Comment 4 Pavel Zilke 2009-11-06 21:26:23 MSK
(В ответ на комментарий №2)
> установил в соотв. как написано в readme, но подцепил к существующей базе.
> (использовал инсталлятор, он распознал что база уже существует и не стал ее
> удалять/создавать/выдавать ошибки)

А в этой базе были записи, содержащие кириллицу?
И если были, то что с ними стало?
Comment 5 Mike Lykov 2009-11-07 09:49:59 MSK
(В ответ на комментарий №3)
> > в интерфейсе - есть ("Последнее обновление:    11/06/2009 14:34:06")
> > в настройках конкр. машины - есть ("Режим работы СборщикаIP ..") (в моем
> > варианте , установленном с сайта тут английский, неполный перевод)
> > в деталях конфигурации  - есть ("Физическая память       A0 (No ECC) ")
> чего и на что нужно исправить?

это я просто отметил, что оно есть, там исправлять не нужно. привел скопированные из интерфейса цитаты.
Comment 6 Mike Lykov 2009-11-07 09:54:17 MSK
> > установил в соотв. как написано в readme, но подцепил к существующей базе.
> > (использовал инсталлятор, он распознал что база уже существует и не стал ее
> > удалять/создавать/выдавать ошибки)
> А в этой базе были записи, содержащие кириллицу?

конечно, записи в "деталях конфигурации" хранятся в базе как я понимаю.

> И если были, то что с ними стало?

вручную на них в базе не смотрел, но из обоих интерфейсов (что из пакета и что установлен вручную) выглядело все одинаково потом.

загадка для меня только в том, что при _первом заходе_ в детали конфигурации компутера через интерфейс из пакета были неверно кодированные знаки в деталях конфигурации (в базе нормально, отображает неверно). вышел. зашел еще раз чтобы скопировать сюда. и их уже не было (стало нормально).

неплохо было бы чтобы еще кто-то протестировал, мне что-то лень всю процедуру установки повторять еще раз чтобы воспроизвести ;)
Comment 7 Pavel Zilke 2009-11-07 11:22:06 MSK
Помнится мне, что как-то напарывался на похожую проблему с кодировкой (только вот не помню, это было после обновления или новой установки). На экране ввода пароля не было надписей "Пользователь", "Пароль". Я щелкнул по значку Российского флага, надписи появились и дальше все работало нормально.
Comment 8 Pavel Zilke 2009-11-26 21:47:04 MSK
Полностью исправлено в 1.02.1-alt2