Skip to content

Предлагаю сделать обязательным ключ "-d" для bootstrap #172

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

Closed
Klink0v opened this issue Oct 14, 2021 · 1 comment
Milestone

Comments

@Klink0v
Copy link

Klink0v commented Oct 14, 2021

Предлагаю сделать обязательным (mandatory) ключ "-d" (выбор базы данных) для операции "mamonsu bootstrap".

Обосновываю.

mamonsu bootstrap по определению должен запускаться от имени суперпользователя базы данных. На большинстве современных дистрибутивов это делается командой типа

sudo -u postgres /usr/bin/mamonsu bootstrap -M mamonsu

При этом соединение с сервером БД происходит через Unix Socket, пароль от пользователя "postgres" не запрашивается.

По умолчанию создание схемы и функций mamonsu происходит в базе данных "по умолчанию", которой для пользователя "postgres" также будет являться "postgres". И при последующем запуска демона mamonsu он совершенно справедливо не найдёт базы данных с именем "mamonsu", а к базе данных "postgres" у него не хватит прав.

Поэтому в подавляющем большинстве случаев использования bootstrap необходимо явно указывать базу данных для mamonsu с ключом "-d". Начинающие же пользователи, как правило, тупо копируют из официальной методички примеры использования, а потом долго не понимают почему у них ничего не работает.

Это не баг mamonsu как таковой, но на usability сказывается очень сильно.

@cuprumtan cuprumtan added this to the 3.1.0 milestone Oct 15, 2021
@cuprumtan
Copy link
Contributor

Добрый день! Это хорошая идея. В новом релизе немного изменили поведение bootstrap: теперь без явного указания параметр dbname автоматически заполняется данными из конфигурационного файла (база, указанная в секции [postgres]). Возможность задать dbname вручную также осталась. Конфигурационный файл можно указать явно, если он не совпадает с дефолтным /etc/mamonsu/agent.conf, с помощью опции -c/--config.

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

No branches or pull requests

2 participants