Bug 5771 - нет информации в теме письма
: нет информации в теме письма
Status: CLOSED FIXED
: Sisyphus
(All bugs in Sisyphus/osec-mailreport)
: unstable
: all Linux
: P1 enhancement
Assigned To:
:
:
:
:
:
  Show dependency tree
 
Reported: 2004-12-23 12:07 by
Modified: 2014-10-23 18:46 (History)


Attachments


Note

You need to log in before you can comment on or make changes to this bug.


Description From 2004-12-23 12:07:55
Хорошо бы иметь настройку для того, чтоб отключать посылку писем, если не было 
изменений
------- Comment #1 From 2004-12-23 15:29:36 -------
1. это небезопасно (если сломают машину и отключат osec, то об этом и не
узнаешь)
2. это уже сделано для сильных духом.
------- Comment #2 From 2004-12-23 19:29:35 -------
(In reply to comment #1) 
> 2. это уже сделано для сильных духом. 
В каком файле? 
 
 
------- Comment #3 From 2006-05-11 16:12:52 -------
Тогда удобно было бы отражать в теме письма
------- Comment #4 From 2006-11-15 12:55:56 -------
Например +2 -3 ~44 означает, что добавлено 2, убрано 3, изменено 44
Так же можно использовать символ ! в случае чего-то очень страшного
------- Comment #5 From 2008-12-18 20:38:09 -------
Все еще достает приблизительно догадываться о пустых письмах по размеру письма
------- Comment #6 From 2009-01-26 18:39:19 -------
перевешиваю на нынешнего мантейнера osec
------- Comment #7 From 2009-02-05 03:02:23 -------
Не думаю, что эта фича востребована.
------- Comment #8 From 2009-02-05 08:55:06 -------
Да это не фича, а бага.
Попробуй принимать такие письма с 10-и машин и читать их все, несмотря на то,
что они пустые. Меня с 2-х достает читать пустые письма.
------- Comment #9 From 2009-04-23 19:53:57 -------
переименую название
------- Comment #10 From 2009-10-09 15:41:20 -------
В сизиф ушёл 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 From 2009-10-09 15:46:28 -------
О! Наконец-то! Спасибо!
------- Comment #12 From 2009-10-11 23:43:27 -------
Может, такой формат и дать по умолчанию?
------- Comment #13 From 2009-10-11 23:46:07 -------
(В ответ на комментарий №12)
> Может, такой формат и дать по умолчанию?

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

Сделал subject:

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

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