git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[BUG] git send-email: incorrectly parses email address with comma

From
Heinrich Schuchardt <xypron.glpk@gmx.de>
Date
May 12, 2018, 08:21 UTC
Message-ID
<9a35041b-8333-9533-4344-08d6ef99c9c9@gmx.de>

Git send-email allows to combine multiple email addresses in one parameter, e.g.

--to="a@example.com, b@example.com"
But email addresses may contain commas themselves:
--to="LASTNAME, firstname <firstname.lastname@example.com>"

This may lead to an error: $ git send-email --to="Schuchardt, Heinrich <xypron.glpk@example.com>" \ 0000*.patch0000-cover-letter.patch (mbox) Adding cc: Heinrich Schuchardt <xypron.glpk@example.com> from line 'From: Heinrich Schuchardt <xypron.glpk@example.com>'

From: Heinrich Schuchardt <xypron.glpk@example.com>
To: Schuchardt,
        Heinrich <xypron.glpk@example.com>
Subject: [PATCH 0/2] efi_loader: adjust definitions of variable services
Date: Sat, 12 May 2018 10:01:21 +0200
Message-Id: <20180512080121.26620-1-xypron.glpk@example.com>
X-Mailer: git-send-email 2.17.0

Send this email? ([y]es|[n]o|[q]uit|[a]ll): a Password for 'smtp://xypron.glpk@example.com@mail.example.com:587': Syntax error in parameters or arguments

Please, implement the following logic,

If the string preceding a comma is not a valid email address do not split it off.

Best regards
Heinrich Schuchardt
Next: Jeff King
Message 1 of 3 in “[BUG] git send-email: incorrectly parses email address with comma”
  1. Heinrich SchuchardtMay 12, 2018
  2. Jeff KingMay 12, 2018
  3. Heinrich SchuchardtMay 12, 2018

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.