Bug#540132: [UDD] has some strange information in the upload_history table
On 09/08/09 at 14:42 -0700, Ryan Niebur wrote:
> On Sun, Aug 09, 2009 at 09:52:23AM +0100, Filippo Giunchedi wrote:
> > On Wed, Aug 05, 2009 at 07:41:17PM -0700, Ryan Niebur wrote:
> > > Package: qa.debian.org
> > > Severity: normal
> > > User: firstname.lastname@example.org
> > > Usertags: udd
> > >
> > > 23:08 < Ryan52> lucas: I was trying to find a list of all the people who have sponsored my uploads, and noticed that UDD's upload_history table has some strange data, could you please take a look at (or tell me w
> > > ho to complain to) the output of: SELECT * FROM upload_history WHERE changed_by_name LIKE 'Ryan Niebur' AND signed_by_name = ''
> > > 23:09 < Ryan52> ;
> > > 23:10 < Ryan52> lucas: oh, the problem with signed_by_name being empty for libopenid-ruby's upload was that a DM sponsored it. so I guess UDD (or wherever UDD gets this data from) does not take into account the D
> > > M keyring and it should?
> > > 23:14 < Ryan52> lucas: hm, and for the crip one, it seems that all uploads signed by phil do that. I'm guessing that because signed_by doesn't include his email address, the (presumably) regexp splitting it into
> > > signed_by_* is parsing it wrong. why it doesn't include his email address, I dunno...
> > > 23:26 * Ryan52 wonders if there's a place where bugs about UDD are supposed to be reported...
> > the upload-history data is taken (for now) from
> > http://qa.debian.org/~filippo/ddc/ and I couldn't find the wrong libopenid-ruby
> > you were talking about, DM keyring should be accounted for
> I was talking about the one that that SQL query shows.
(To clarify: Filippo wrote a parser for debian-devel-changes@, and
generates a text summary that I import in UDD. That's why it would be
more helpful to provide more info about the package in question).
> > $ zgrep -A12 libopenid-ruby * | grep -i signed
> > debian-devel-changes.200712.gz:Signed-By: Paul van Tilburg <email@example.com>
> > debian-devel-changes.200803.gz:Signed-By: Antonio Terceiro <firstname.lastname@example.org>
> > debian-devel-changes.200807.gz:Signed-By: Antonio Terceiro <email@example.com>
> > debian-devel-changes.200905.gz:Signed-By: Antonio Terceiro <firstname.lastname@example.org>
> > debian-devel-changes.200907.gz:Signed-By: Lucas Nussbaum <email@example.com>
> > for the other problem you are correct, the first uid for ED1A3933B2CFCDD8
> > doesn't contain an email:
> > $ gpg --list-keys ED1A3933B2CFCDD8
> > pub 1024D/B2CFCDD8 2003-01-08
> > uid Philipp Kern
> > uid Philipp Kern <firstname.lastname@example.org>
> > uid Philipp Kern <email@example.com>
> > uid Philipp Kern <firstname.lastname@example.org>
> > uid Philipp Kern <email@example.com>
> > uid Philipp Kern <firstname.lastname@example.org>
> > I'm not sure how to fix this though, I'm relying on gpgv output
> well the parsing is clearly broken... the name should be "Philipp
> Kern" and the email address should be empty.
The split between name and email is not by the UDD importer using
email.Utils.parseaddr (Python). Apparently, it doesn't like being asked
to parse a string without an email.
The Signed-by columns are mostly provided for convenience. If you really
want to know who signed an upload, use the key together with
carnivore_keys to find out.
| Lucas Nussbaum
| email@example.com http://www.lucas-nussbaum.net/ |
| jabber: firstname.lastname@example.org GPG: 1024D/023B3F4F |