Re: [PATCH b4] Fix broken fallback to user.email

[Dave Marquardt <davemarq@linux.ibm.com>]

Jonas Rebmann <jre@pengutronix.de> writes:

> After conversion to string, fromaddr becomes "None" as a string literal,
> so the user.email fallback would never be reached if sendemail.from is
> unset, eventually leading to `smtplib.SMTPSenderRefused` (<None>: sender
> address must contain a domain').
>
> Convert to string after the check.
>
> Fixes: 742e017c1b ("Cleanup: initial round addressing typing issues reported by mypy")
> Signed-off-by: Jonas Rebmann <jre@pengutronix.de>
> ---
>  src/b4/__init__.py | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/src/b4/__init__.py b/src/b4/__init__.py
> index ff6075e..c116344 100644
> --- a/src/b4/__init__.py
> +++ b/src/b4/__init__.py
> @@ -3949,8 +3949,10 @@ def get_sendemail_config() -> Dict[str, Optional[Union[str, List[str]]]]:
>  def get_smtp(dryrun: bool = False) -> Tuple[Union[smtplib.SMTP, smtplib.SMTP_SSL, List[str], None], str]:
>      sconfig = get_sendemail_config()
>      # Limited support for smtp settings to begin with, but should cover the vast majority of cases
> -    fromaddr = str(sconfig.get('from'))
> -    if not fromaddr:
> +    fromaddr = sconfig.get('from')
> +    if fromaddr:
> +        fromaddr = str(fromaddr)
> +    else:
>          # We fall back to user.email
>          usercfg = get_user_config()
>          fromaddr = str(usercfg['email'])

Reviewed-by: davemarq@linux.ibm.com