Submitter is not detected/created for some patches

[Paul Sokolovsky]

For example http://patches.linaro.org/19517/ . As of now it has:

---------------
Submitter
Subject [Linaro-mm-sig,v6,3/4,UPDATED] drivers: of: add initialization code for dma reserved memory

 - Aug. 23, 2013, 8:39 a.m.

From: Marek Szyprowski <email address hidden>

This patch adds device tree support for contiguous and reserved memory
regions defined in device tree.
---------------

I.e. both submitter and first comment (i.e. original message) author fields are empty. At the same time, it's clear that well-formed email is just there.

[Paul Sokolovsky (pfalcon)]

We had issues with such patches with empty submitter being randomly associated to newly logged in crowd user, see https://code.launchpad.net/~pfalcon/linaro-patchmetrics/fix-empty-email/+merge/191831

[Paul Sokolovsky (pfalcon)]

It's unclear who exactly owns such empty-submitter patches (submitter_id can't be null), but apparently it's:

 id | username | first_name | last_name | email | password | is_staff | is_active | is_superuser | last_login | date_joined
-----+----------------------------+------------+-----------+-------+----------+----------+-----------+--------------+-------------------------------+-------------------------------
 204 | openiduser2 | | E:V:A | | ! | f | t | f | 2012-03-11 23:44:07.673939+00 | 2012-03-11 23:44:07.503501+00

(can't verify on staging.patches as of now as I had patches the DB there.)

FWIW, not counting that patch (which is re-owned by another user now on staging):

# select id from patchwork_patch where submitter_id=204;
 id
----
(0 rows)

[Milo Casagrande (milo)]

Using Crowd as the authentication form for sent patches, a patch cannot be accepted if it does not have a valid email address.
I'm closing this one now, in case reopen with as much details as possible using https://bugs.linaro.org.