Re: [devel] [git update] packages/prometeus: heads/master

Re: [devel] [git update] packages/prometeus: heads/master

[Slava Semushin]

07.09.07, Avramenko Andrew<liks / altlinux.org> написал(а):
> Update of /people/liks/packages/prometeus.git
[...]
> --- a/prometeus/INSTALL
> +++ b/prometeus/INSTALL
> +    * dbinfo - данные для соединения с базой. Имеет права root:root, потому что
> +        содержит имя пользователя и пароль.
[...]

"Имеет права root:root" IMHO не очень корректно. В данном случае
"имеет владельца и группу root:root" (хотя это тоже не лучший
вариант), а права это относится к маске, типа 644 или 755.

Кстати, то что файлов владеет рут и группа рут мало поможет файлу,
если у него выставлены атрибуты на чтение для всех остальных :)
Короче, желательно на этот файл поставить права 600 или 640 --
как-нить так.

Надо об этом написать в INSTALL, наверное, ну и в RPM-пакете тоже про
права не забыть.

> diff --git a/prometeus/scripts/dbinfo b/prometeus/scripts/dbinfo
> new file mode 100644
> index 0000000..d13cc51
> --- /dev/null
> +++ b/prometeus/scripts/dbinfo
> @@ -0,0 +1,11 @@
> +#!/usr/bin/perl
> +
> +# Name of database
> +our $db     = "prometeus";
> +# Name of MySQL user
> +our $dbUser = "prometeus";
> +# User Password
> +our $dbPass = "qwerty";
> +# Host of MySQL
> +our $dbHost = "localhost";
> +
[...]

-- 
+ Slava Semushin | slava.semushin @ gmail.com
+ ALT Linux Team | php-coder @ altlinux.ru