Bug 5771 - нет информации в теме письма
Summary: нет информации в теме письма
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: osec-mailreport (show other bugs)
Version: unstable
Hardware: all Linux
: P1 enhancement
Assignee: Alexey Gladkov
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-12-23 12:07 MSK by Sergey V Turchin
Modified: 2014-10-23 18:46 MSK (History)
6 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Sergey V Turchin 2004-12-23 12:07:55 MSK
Хорошо бы иметь настройку для того, чтоб отключать посылку писем, если не было 
изменений
Comment 1 inger@altlinux.org 2004-12-23 15:29:36 MSK
1. это небезопасно (если сломают машину и отключат osec, то об этом и не узнаешь)
2. это уже сделано для сильных духом.
Comment 2 Sergey V Turchin 2004-12-23 19:29:35 MSK
(In reply to comment #1) 
> 2. это уже сделано для сильных духом. 
В каком файле? 
 
 
Comment 3 Sergey V Turchin 2006-05-11 16:12:52 MSD
Тогда удобно было бы отражать в теме письма
Comment 4 Sergey V Turchin 2006-11-15 12:55:56 MSK
Например +2 -3 ~44 означает, что добавлено 2, убрано 3, изменено 44
Так же можно использовать символ ! в случае чего-то очень страшного
Comment 5 Sergey V Turchin 2008-12-18 20:38:09 MSK
Все еще достает приблизительно догадываться о пустых письмах по размеру письма
Comment 6 inger@altlinux.org 2009-01-26 18:39:19 MSK
перевешиваю на нынешнего мантейнера osec
Comment 7 Alexey Gladkov 2009-02-05 03:02:23 MSK
Не думаю, что эта фича востребована.
Comment 8 Sergey V Turchin 2009-02-05 08:55:06 MSK
Да это не фича, а бага.
Попробуй принимать такие письма с 10-и машин и читать их все, несмотря на то, что они пустые. Меня с 2-х достает читать пустые письма.
Comment 9 Sergey V Turchin 2009-04-23 19:53:57 MSD
переименую название
Comment 10 Alexey Gladkov 2009-10-09 15:41:20 MSD
В сизиф ушёл osec-1.2.4 в котором можно включить summary об изменениях.
В pipe.conf теперь можно указывать:

MAIL_PIPE='/bin/mail -s "$HOSTNAME: +$ADDED -$REMOVED ~$CHANGED" root'

или

MAIL_PIPE='/bin/mail -s "$HOSTNAME: $STAT" root'

что будет эквивалентно:

MAIL_PIPE='/bin/mail -s "$HOSTNAME: chg=$CHANGED,add=$ADDED,del=$REMOVED" root'
Comment 11 Sergey V Turchin 2009-10-09 15:46:28 MSD
О! Наконец-то! Спасибо!
Comment 12 Michael Shigorin 2009-10-11 23:43:27 MSD
Может, такой формат и дать по умолчанию?
Comment 13 Alexey Gladkov 2009-10-11 23:46:07 MSD
(В ответ на комментарий №12)
> Может, такой формат и дать по умолчанию?

В конфиге есть готовые примеры. Нужно только раскомментировать при наличии пакета.
Comment 14 Zerg 2009-10-11 23:52:29 MSD
(В ответ на комментарий №12)
> Может, такой формат и дать по умолчанию?
Я хотел проверить, прежде, чем это написать :-)
Comment 15 Zerg 2009-10-11 23:53:33 MSD
(В ответ на комментарий №13)
> Нужно только раскомментировать при наличии пакета.
Чем может быть плохо раскомментировать по умолчанию ?
Comment 16 Zerg 2009-10-13 21:57:03 MSD
не работает
Comment 17 Zerg 2009-10-13 21:58:08 MSD
/etc/cron.daily/osec: line 72: ADDED: unbound variable
Comment 18 Zerg 2009-10-13 22:02:54 MSD
Предлагаю по умолчанию
MAIL_PIPE='/bin/mail -s "[osec] $HOSTNAME: +$ADDED -$REMOVED ~$CHANGED" root'
или
MAIL_PIPE='/bin/mail -s "[osec] $HOSTNAME: $HUMAN_READABLE_STAT" root'
что будет эквивалентно:
MAIL_PIPE='/bin/mail -s "[osec] $HOSTNAME: +$ADDED -$REMOVED ~$CHANGED" root'
Comment 19 Michael Shigorin 2009-10-13 23:54:37 MSD
(In reply to comment #18)
> Предлагаю по умолчанию
> MAIL_PIPE='/bin/mail -s "[osec] $HOSTNAME: +$ADDED -$REMOVED ~$CHANGED" root'
Угу, эта форма более прозрачна.
Comment 20 Alexey Gladkov 2009-10-14 00:03:36 MSD
(В ответ на комментарий №19)
> (In reply to comment #18)
> > Предлагаю по умолчанию
> > MAIL_PIPE='/bin/mail -s "[osec] $HOSTNAME: +$ADDED -$REMOVED ~$CHANGED" root'
> Угу, эта форма более прозрачна.

Для меня "~" не очевиден.
Comment 21 Zerg 2009-10-14 00:06:56 MSD
(В ответ на комментарий №20)
> (В ответ на комментарий №19)
> Для меня "~" не очевиден.
Не проблема. Возьми любой другой 1 символ

Так же, поля с нулями в тему вносить не нужно.
А если одни нули можно просто фразу типа "no changed found", тогда еще лучше читаться будет.
Comment 22 Alexey Gladkov 2009-10-14 00:24:45 MSD
Мне больше нравится chg=5,add=3,del=8
Comment 23 Michael Shigorin 2009-10-14 00:27:18 MSD
(In reply to comment #22)
> Мне больше нравится chg=5,add=3,del=8
Может, тогда "5 changed, 3 added, 8 deleted"? (или хотя бы с пробелами после запятых, чтоб глаз в буковках не завязал)
Comment 24 Alexey Gladkov 2009-10-14 00:33:53 MSD
(В ответ на комментарий №23)
> Может, тогда "5 changed, 3 added, 8 deleted"? (или хотя бы с пробелами после
> запятых, чтоб глаз в буковках не завязал)

Если у меня получится, то ты сможешь сделать как захочешь.
Comment 25 Alexey Gladkov 2009-10-14 00:43:53 MSD
(В ответ на комментарий №17)
> /etc/cron.daily/osec: line 72: ADDED: unbound variable

Это потому, что ты не сменил SEND_PIPE.
Я знаю, что не очевидно получилось.

Попробую сделать очевиднее.
Comment 26 Alexey Gladkov 2009-10-14 03:08:35 MSD
(В ответ на комментарий №23)
> Может, тогда "5 changed, 3 added, 8 deleted"? (или хотя бы с пробелами после
> запятых, чтоб глаз в буковках не завязал)

Сделал 1.2.4-alt2, там есть возможность задавать CHANGED, ADDED, REMOVED и STAT в обоих скриптах: osec.cron и osec_mailer.

Миш, если ты захочешь, то сможешь сделать любой заголовок, используя первые три переменных. Думаю, это честно.
Comment 27 Alexey Gladkov 2009-10-14 03:31:31 MSD
Исправлено в 1.2.4-alt2.
Comment 28 Zerg 2013-04-03 14:04:35 MSK
Так, все-таки, почему бы не сделать заголовок хоть немного информативным?
Comment 29 Alexey Gladkov 2013-04-03 14:06:15 MSK
Оно уже информативно.
Comment 30 Zerg 2013-04-03 14:11:52 MSK
Не вижу ничего нового в pipe.conf
Какая новая информация теперь есть в теме письма?
Comment 31 Alexey Gladkov 2013-04-03 14:20:55 MSK
(В ответ на комментарий №30)
> Не вижу ничего нового в pipe.conf
> Какая новая информация теперь есть в теме письма?

Как я писал ранее, в pipe.conf теперь можно указывать:

MAIL_PIPE='/bin/mail -s "$HOSTNAME: +$ADDED -$REMOVED ~$CHANGED" root'

или

MAIL_PIPE='/bin/mail -s "$HOSTNAME: $STAT" root'

что будет эквивалентно:

MAIL_PIPE='/bin/mail -s "$HOSTNAME: chg=$CHANGED,add=$ADDED,del=$REMOVED" root'

Учитывая, что на вкус и цвет согласия нет, то каждый может сформировать заголовок для себя сам.
Comment 32 Sergey V Turchin 2013-04-03 15:01:49 MSK
(В ответ на комментарий №31)
> каждый может сформировать заголовок для себя сам.
Это и так понятно.

Почему бы по умолчанию не сделать заголовок хоть немного информативным?
Comment 33 Alexey Gladkov 2013-04-03 15:07:17 MSK
(В ответ на комментарий №32)
> Почему бы по умолчанию не сделать заголовок хоть немного информативным?

Хорошо. В следующей версии я сделаю по умолчанию: "$HOSTNAME: $STAT"
Comment 34 Sergey V Turchin 2013-04-03 16:19:55 MSK
(В ответ на комментарий №33)
> Хорошо. В следующей версии я сделаю по умолчанию: "$HOSTNAME: $STAT"
Клево, спасибо!
На сём я полностью буду удовлетворен.
Comment 35 Sergey Y. Afonin 2014-10-22 11:43:20 MSK
(In reply to comment #33)
> (В ответ на комментарий №32)
> > Почему бы по умолчанию не сделать заголовок хоть немного информативным?
> 
> Хорошо. В следующей версии я сделаю по умолчанию: "$HOSTNAME: $STAT"

Переоткрою. Чуть-чуть бы доделать. Надо ещё переменную, скажем, $STATE c состояниями "PASSED и FAILED". Чтобы по ней сортировать можно было по заголовку. Можно, конечно, нагородить огород с регулярным выражением по нахождению некоторого количества нулей в выводе $STAT, но не очень удобно. И умолчание сделать "[osec] Daily security check $STATE -- $HOSTNAME".
Comment 36 Michael Shigorin 2014-10-23 13:54:31 MSK
(В ответ на комментарий №35)
> Переоткрою. Чуть-чуть бы доделать.
Не надо так делать -- у каждой хорошей баги должны быть чёткие начало, формулировка и конец.  Когда начинаются бесконечные самоуточняющиеся портянки, незаметно растёт наклад времени на их разбор и "diff" для выяснения, что ж там ещё; теряют точность ссылки на багу в %changelog (или же приходится сопоставлять и время); многих подобное подвигает в сторону "забить".

Проверено на случае, где "портянки" дошли до клинических и в итоге пришлось писать внутренний регламент.

Есть отдельная проблема -- есть отдельная бага.

Добавил на http://www.altlinux.org/BugTracking/BugzillaMiniHowto на всякий...
Comment 37 Sergey V Turchin 2014-10-23 15:58:07 MSK
(В ответ на комментарий №36)
> Есть отдельная проблема -- есть отдельная бага.
Согласен.

P.S.
Куча проблем сваленных в один баг -- бардак, переоткрытия, переименования, лебедь-рак-щука и т.д.
Comment 38 Sergey Y. Afonin 2014-10-23 17:32:45 MSK
Вот ведь накинулись, а... :-) Bug 30414

Я сам не люблю всё в один баг мешать и, частенько, занимаюсь растаскиванием одного бага на несколько. Но тут уж очень в тему: по факту баг не закрыт, так как, по-умолчанию, информации в теме всё ещё нет.
Comment 39 Sergey V Turchin 2014-10-23 18:33:16 MSK
По факту и большому опыту забить проще ;-)
Comment 40 Alexey Gladkov 2014-10-23 18:46:49 MSK
(В ответ на комментарий №35)
> Можно, конечно, нагородить огород с регулярным выражением по
> нахождению некоторого количества нулей в выводе $STAT, но не очень удобно. И
> умолчание сделать "[osec] Daily security check $STATE -- $HOSTNAME".

Сделал subject:

[osec${PROFILE:+:$PROFILE}] Daily security check ($STAT) -- $HOSTNAME" root

по умолчанию.