Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the CGI CPAN distribution.

Report information
The Basics
Id: 49630
Status: resolved
Priority: 0/
Queue: CGI
People
Owner: MARKSTOS [...] cpan.org
Requestors: mkanat [...] cpan.org
yuri [...] rawbw.com
Cc:
AdminCc:
Bug Information
Severity: Critical
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Fri Sep 11 12:46:11 2009 mkanat [...] cpan.org - Ticket created
Subject: Chagnes to CGI::Carp in 3.47 break TT and exception handling
CGI::Carp now does this before ever calling realdie, even if we're in an eval: # if $arg is a reference, give it a chance to # be stringified $arg = "$arg" if ref $arg; $arg = join '' => $arg, @rest ; That turns all object exceptions into strings, and re-throws them as strings. This breaks everything that depends on object-based exceptions, including Template Toolkit. (We noticed this because it causes Bugzilla to entirely not function correctly.)
  Fri Sep 11 12:53:09 2009 mark [...] summersault.com - Correspondence added
Subject: Re: [rt.cpan.org #49630] Chagnes to CGI::Carp in 3.47 break TT and exception handling
Date: Fri, 11 Sep 2009 12:52:49 -0400
To: bug-CGI.pm [...] rt.cpan.org
From: Mark Stosberg <mark [...] summersault.com>
Show quoted text
> CGI::Carp now does this before ever calling realdie, even if we're in an > eval: > > # if $arg is a reference, give it a chance to > # be stringified > $arg = "$arg" if ref $arg; > > $arg = join '' => $arg, @rest ; > > That turns all object exceptions into strings, and re-throws them as > strings. This breaks everything that depends on object-based exceptions, > including Template Toolkit. (We noticed this because it causes Bugzilla > to entirely not function correctly.)
Thanks for the report. We can see about backing out this change and looking at different approaches for solving the bug which introduced it. Mark
  Fri Sep 11 12:53:10 2009 The RT System itself - Status changed from 'new' to 'open'
  Fri Sep 11 21:38:15 2009 MARKSTOS [...] cpan.org - Taken
  Fri Sep 11 21:38:24 2009 MARKSTOS [...] cpan.org - Reference to ticket #45956 added
  Fri Sep 11 21:40:43 2009 MARKSTOS [...] cpan.org - Reference to ticket #45956 deleted
  Fri Sep 11 21:40:43 2009 MARKSTOS [...] cpan.org - Reference to ticket #41530 added
  Fri Sep 11 21:42:21 2009 MARKSTOS [...] cpan.org - Reference to ticket #41530 deleted
  Fri Sep 11 21:42:21 2009 MARKSTOS [...] cpan.org - Reference to ticket #39904 added
  Fri Sep 11 21:49:47 2009 MARKSTOS [...] cpan.org - Correspondence added
On Fri Sep 11 12:46:11 2009, MKANAT wrote: Show quoted text
> CGI::Carp now does this before ever calling realdie, even if we're in an > eval: > > # if $arg is a reference, give it a chance to > # be stringified > $arg = "$arg" if ref $arg; > > $arg = join '' => $arg, @rest ; > > That turns all object exceptions into strings, and re-throws them as > strings. This breaks everything that depends on object-based exceptions, > including Template Toolkit. (We noticed this because it causes Bugzilla > to entirely not function correctly.)
This is now patched in my github repo now. Thanks for the report. Mark
  Fri Sep 11 21:49:49 2009 MARKSTOS [...] cpan.org - Status changed from 'open' to 'patched'
  Thu Sep 24 22:23:00 2009 yuri [...] rawbw.com - Ticket #50007: Ticket created
Subject: Certain latest versions of CGI (CGI-3.47) break Bugzilla-3.4.2
Date: Thu, 24 Sep 2009 19:22:31 -0700
To: bug-CGI.pm [...] rt.cpan.org
From: Yuri <yuri [...] rawbw.com>
Bugzilla's submit bug script produces a message: file error - undef error - undef error - file error - undef error - undef error - file error - undef error - return error - at lib/x86_64- linux-thread-multi/Template/Context.pm line 508. See this discussion for example: http://groups.google.com/group/mozilla.support.bugzilla/browse_thread/thread/9fbce9eecac6d8de/0c7d1dd8a7ec9341?lnk=raot Yuri
  Fri Sep 25 09:52:07 2009 MARKSTOS [...] cpan.org - Ticket #50007: Merged into ticket #49630
  Fri Sep 25 09:52:07 2009 MARKSTOS [...] cpan.org - Merged into ticket #49630
  Fri Sep 25 09:52:25 2009 MARKSTOS [...] cpan.org - Subject changed from 'Chagnes to CGI::Carp in 3.47 break TT and exception handling' to 'Chagnes to CGI::Carp in 3.47 break TT and exception handling (Breaks BugZilla)'
  Fri Sep 25 09:54:53 2009 MARKSTOS [...] cpan.org - Correspondence added
RT-Send-CC: lds [...] cpan.org
Lincoln, I recommend we make a new release to address this regression which affected Template Toolkit and Bugzilla. A patch for this is already in github, and linked from this ticket history. I think the general state of github is safe for a new release. The potential modifications relating to PSGI are intentionally on a different branch. Mark
  Fri Sep 25 09:54:53 2009 The RT System itself - Status changed from 'patched' to 'open'
  Fri Sep 25 11:07:31 2009 LDS [...] cpan.org - Correspondence added
Fix now released in 3.48.
  Fri Sep 25 11:07:34 2009 LDS [...] cpan.org - Status changed from 'open' to 'resolved'
  Sat Oct 03 20:15:17 2009 MARKSTOS [...] cpan.org - Subject changed from 'Chagnes to CGI::Carp in 3.47 break TT and exception handling (Breaks BugZilla)' to 'Changes to CGI::Carp in 3.47 break TT and exception handling (Breaks BugZilla)'
  Sat Oct 03 20:17:17 2009 MARKSTOS [...] cpan.org - Correspondence added
Could you test an alternate fix to this for us in combination with Bugzilla? http://github.com/markstos/CGI.pm/raw/master/lib/CGI/Carp.pm We'd like to confirm the new approach works *before* the next release, rather than afterwards. The goal continue to handle the Bugzilla cases appropriately, as well as some addressing some related bugs. It was attempt to fix another problem which is what triggered the regression. The proposed fix should ideally cover both cases. Mark
  Sat Oct 03 20:17:18 2009 The RT System itself - Status changed from 'resolved' to 'open'
  Sat Oct 03 20:33:01 2009 mkanat [...] cpan.org - Correspondence added
On Sat Oct 03 20:17:17 2009, MARKSTOS wrote: Show quoted text
> Could you test an alternate fix to this for us in combination with > Bugzilla? > > http://github.com/markstos/CGI.pm/raw/master/lib/CGI/Carp.pm > [snip]
Hey Mark! Thanks for thinking of us! :-) Yes, it definitely works. Just as a question, though--why not put the ineval() bit right after the DIE_HANDLER and just pass it ($arg, @rest)? Seems like that would be the most reliable way to make sure that you don't break apps that rely on any eval-related behavior of standard die().
  Sat Oct 03 20:44:50 2009 MARKSTOS [...] cpan.org - Correspondence added
RT-Send-CC: yanick
Yanick, Could you address this feedback? Mark On Sat Oct 03 20:33:01 2009, MKANAT wrote: Show quoted text
> On Sat Oct 03 20:17:17 2009, MARKSTOS wrote:
> > Could you test an alternate fix to this for us in combination with > > Bugzilla? > > > > http://github.com/markstos/CGI.pm/raw/master/lib/CGI/Carp.pm > > [snip]
> > Hey Mark! Thanks for thinking of us! :-) > > Yes, it definitely works. > > Just as a question, though--why not put the ineval() bit right after > the DIE_HANDLER and just pass it ($arg, @rest)? Seems like that would be > the most reliable way to make sure that you don't break apps that rely > on any eval-related behavior of standard die().
  Sun Oct 04 12:08:04 2009 yanick [...] babyl.dyndns.org - Correspondence added
Subject: Re: [rt.cpan.org #49630] Changes to CGI::Carp in 3.47 break TT and exception handling (Breaks BugZilla)
Date: Sun, 04 Oct 2009 12:10:44 -0400
To: bug-CGI.pm [...] rt.cpan.org
From: Yanick Champoux <yanick [...] babyl.dyndns.org>
MARKSTOS via RT wrote: Show quoted text
> Yanick, > > Could you address this feedback?
With great pleasure. :-) Show quoted text
>> Just as a question, though--why not put the ineval() bit right after >> the DIE_HANDLER and just pass it ($arg, @rest)? Seems like that would be >> the most reliable way to make sure that you don't break apps that rely >> on any eval-related behavior of standard die().
If I understand your suggestion correctly, you're proposing to do something akin to sub die { my ($arg,@rest) = @_; &$DIE_HANDLER($arg,@rest) if $DIE_HANDLER; realdie $arg, @rest if ineval(); # and then the other stuff... The problem with that is that if you do eval { CGI::Carp::die( "I died" ) } You'll get I died at CGI/Carp.pm line 354. instead of I died at your/program.cgi line 123. All the gymnastic before the call to 'realdie' is to ensure that the call to CGI::Carp::die will be transparent and magically yield the same useful information than the real 'die'. Makes sense? :-) Joy, `/anick
  Sun Oct 04 13:56:55 2009 mkanat [...] cpan.org - Correspondence added
On Sun Oct 04 12:08:04 2009, yanick@babyl.dyndns.org wrote: Show quoted text
> [snip] > Makes sense? :-)
Ah yeah, makes total sense. :-) Thanks for the explanation!
  Sun Oct 04 17:00:26 2009 yanick [...] babyl.dyndns.org - Correspondence added
Subject: Re: [rt.cpan.org #49630] Changes to CGI::Carp in 3.47 break TT and exception handling (Breaks BugZilla)
Date: Sun, 04 Oct 2009 17:03:17 -0400
To: bug-CGI.pm [...] rt.cpan.org
From: Yanick Champoux <yanick [...] babyl.dyndns.org>
Max Kanat-Alexander via RT wrote: Show quoted text
> On Sun Oct 04 12:08:04 2009, yanick@babyl.dyndns.org wrote:
>> [snip] >> Makes sense? :-)
> > Ah yeah, makes total sense. :-) Thanks for the explanation!
My pleasure. And thanks for the feedback and the suggestion! Joy, `/anick
  Fri Nov 11 22:14:25 2011 MARKSTOS [...] cpan.org - Correspondence added
This was resolved some time ago. It just needed to be closed.
  Fri Nov 11 22:14:26 2011 MARKSTOS [...] cpan.org - Status changed from 'open' to 'resolved'
  Fri May 23 14:29:33 2014 The RT System itself - Queue changed from CGI.pm to CGI