Bug #102988 for App-cpanminus-reporter: Missing from field (?) in reports

Sun Mar 22 06:00:35 2015 SREZIC [...] cpan.org - Ticket created

Subject:

Missing from field (?) in reports

Reports generated with App-cpanminus-reporter seem to miss the from/tester name field somewhere. The effect is that the "fast" cpantesters matrix view cannot display the tester name for reports generated with App-cpanminus-reporter. See http://217.199.168.174/cgi-bin/cpantestersmatrix.pl?dist=DateTime%201.18;reports=1#sl=7,1 for a sample. The "fast" cpantesters matrix view is using the log file accesible at http://metabase.cpantesters.org/tail/log.txt to generate the table contents. So probably the tester name is missing there, too.

Sun Mar 22 14:38:54 2015 ether [...] cpan.org - Correspondence added

On 2015-03-22 03:00:35, SREZIC wrote: Show quoted text

> Reports generated with App-cpanminus-reporter seem to miss the > from/tester name field somewhere. The effect is that the "fast" > cpantesters matrix view cannot display the tester name for reports > generated with App-cpanminus-reporter. See > http://217.199.168.174/cgi- > bin/cpantestersmatrix.pl?dist=DateTime%201.18;reports=1#sl=7,1 for a > sample. > > The "fast" cpantesters matrix view is using the log file accesible at > http://metabase.cpantesters.org/tail/log.txt to generate the table > contents. So probably the tester name is missing there, too.

I'm confused - I see a tester name for all the reports at that link? It's there for my reports too, and all my reports come from cpanm-reporter.

Sun Mar 22 14:38:54 2015 The RT System itself - Status changed from 'new' to 'open'

Sun Mar 22 14:58:51 2015 SREZIC [...] cpan.org - Correspondence added

On 2015-03-22 14:38:54, ETHER wrote: Show quoted text

> On 2015-03-22 03:00:35, SREZIC wrote:

> > Reports generated with App-cpanminus-reporter seem to miss the > > from/tester name field somewhere. The effect is that the "fast" > > cpantesters matrix view cannot display the tester name for reports > > generated with App-cpanminus-reporter. See > > http://217.199.168.174/cgi- > > bin/cpantestersmatrix.pl?dist=DateTime%201.18;reports=1#sl=7,1 for a > > sample. > > > > The "fast" cpantesters matrix view is using the log file accesible at > > http://metabase.cpantesters.org/tail/log.txt to generate the table > > contents. So probably the tester name is missing there, too.

> > > I'm confused - I see a tester name for all the reports at that link?

The log.txt view shows only the last 1000 reports, so you need luck to catch one. Right now there are some candidates, e.g.: [2015-03-22T18:45:16Z] [] [pass] [DAGOLDEN/Parse-CPAN-Meta-1.4414.tar.gz] [x86_64-linux] [perl-v5.16.3] [9543cda2-d0c3-11e4-a14f-22c2c615bc7f] [2015-03-22T18:45:16Z] But it should look like this (2nd field with name): [2015-03-22T18:45:13Z] [Chris Williams (BINGOS)] [pass] [PERLANCAR/App-IODUtils-0.05.tar.gz] [amd64-netbsd] [perl-v5.20.2] [935f2fcc-d0c3-11e4-b221-9e126cbd7f71] [2015-03-22T18:45:13Z] Show quoted text

> It's there for my reports too, and all my reports come from cpanm-reporter.

It seems that your reports are not affected, so maybe it's a configuration issue?

Sat Apr 23 15:51:20 2016 garu [...] cpan.org - Correspondence added

It looks like a configuration issue indeed. That data is apparently taken from the "email_from" field, where you can put from just an email to the complete name. It is however not mandatory, since no email is actually sent, so people might simply remove it, in which case it will send an empty string. If you think that's not good enough (I know I don't) please come see me so we can talk to David Golden and maybe change that on CPAN::Reporter and CPAN::Testers::Common::Client, where that information is actually validated. Thanks!

Sat Apr 23 15:51:21 2016 garu [...] cpan.org - Status changed from 'open' to 'rejected'