Open Bug 1342813 Opened 7 years ago Updated 7 years ago

$SIG{__DIE__} is set to Bugzilla::_console_die() when running under Plack

Categories

(Bugzilla :: Bugzilla-General, defect)

5.1.1
defect
Not set
normal

Tracking

()

Bugzilla 6.0

People

(Reporter: dylan, Assigned: dylanAtHome)

Details

Attachments

(3 files)

Attached file what-is-die.cgi
I suspect we need to set $BZ_PLACK much earlier in app.psgi, else we think we're in command line mode and set a DIE handler to _console_die which breaks plack's error reporting.

I attached the .cgi I wrote to debug this.
Assignee: general → dylan
Target Milestone: --- → Bugzilla 6.0
Fixed it! Now we get beautiful Plack stack traces when running in development mode.

For details, see:
https://metacpan.org/pod/Plack::Middleware::StackTrace
Assignee: dylan → dylan
Comment on attachment 8841379 [details] [review]
[bugzilla] dylanwh:bug-1342813 > bugzilla:master

Last one. This is the last one in my series of quick fixups that touches app.psgi. You could actually review this and transfer the r+ to the other branches now that I think about it. The choice is yours.
Attachment #8841379 - Flags: review?(jfearn)
Comment on attachment 8841379 [details] [review]
[bugzilla] dylanwh:bug-1342813 > bugzilla:master

Unless for init_page call needs a condition or needs to be removed.
Attachment #8841379 - Flags: review?(jfearn) → review-
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: