Bug#991781: RFR: fail2ban can't send e-mail using mail from bsd-mailx

[Justin B Rye <justin.byam.rye@gmail.com>]

Paul Gevers wrote:
> I have the attached patch ready to push. I'm wondering if we need more
> detailed instructions.
>  
> +  <section id="fail2ban">
> +    <title>fail2ban can't send e-mail using mail from bsd-mailx</title>
> +    <para>
> +      The <systemitem role="package">fail2ban</systemitem> package can
> +      be configured to send out e-mail notifications. It does that
> +      using <command>mail</command>, which is provided by multiple
> +      packages in Debian. A security update (needed on systems that

Has bullseye added extra providers for mailx?  On buster there's
only bsd-mailx and mailutils.  Mind you, fail2ban only Suggests
mailx, so maybe it's assuming some more complicated alternative.

> +      use <command>mail</command> from <systemitem
> +      role="package">mailutils</systemitem>) just before the release
> +      of bullseye broke this functionality for systems that have
> +      <command>mail</command> provided by <systemitem
> +      role="package">bsd-mailx</systemitem>. Users of
> +      <systemitem>fail2ban</systemitem> in combination with

Don't these need to be <systemitem role="package">? I'm not sure what
<systemitem> on its own does.

> +      <systemitem>bsd-mailx</systemitem> that wish

English fix: I'd prefer                     who wish

> +      <systemitem>fail2ban</systemitem> to send out e-mail, should

English fix: better without a comma here                     ^

> +      either switch to a different provide for <command>mail</command>

English fix: typo                     provider

(but if there's no other provider, we might as well say "switch
to mailutils")

> +      or manually unapply <ulink
> +      url="https://github.com/fail2ban/fail2ban/commit/410a6ce5c80dd981c22752da034f2529b5eee844";>the
> +      upstream commit</ulink> (all files are in
> +      <filename>/etc/fail2ban/action.d/</filename>).
> +    </para>
> +  </section>
> +

A suggestion for a slightly more detailed version:

         or manually unapply <ulink
         url="https://github.com/fail2ban/fail2ban/commit/410a6ce5c80dd981c22752da034f2529b5eee844";>the
         upstream commit</ulink> (which inserted the string
	 "<literal>-E 'set escape'</literal>" in multiple places under
         <filename>/etc/fail2ban/action.d/</filename>).

-- 
JBR	with qualifications in linguistics, experience as a Debian
	sysadmin, and probably no clue about this particular package