Status

()

RESOLVED FIXED
6 months ago
6 months ago

People

(Reporter: dylan, Assigned: dylan)

Tracking

Production

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 months ago
Posted file GitHub Pull Request
There are some things that should've been in the first patch but were missed:

1. Calling $c->finish in the finally block should not happen if an exception has been raised.
2. Bugzilla->cleanup() should be called at the same time the mojolicious stash is cleared.
3. Code referencing the shutdownhtml should be removed
4. The conditionals that ran code in Bugzilla.pm when it was not run under mod_perl should instead check where the Bugzilla.pm module was loaded from.
5. Revert the default template change
(Assignee)

Comment 1

6 months ago
Also removed some stuff that manipulates the PATH and signals, which we shouldn't do
(Assignee)

Updated

6 months ago
Status: NEW → RESOLVED
Last Resolved: 6 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.