Opened 16 years ago
Closed 16 years ago
#4514 closed patch (fixed)
mythweb.pl only loads handler.pl once when using mod_perl
Reported by: | Owned by: | xris | |
---|---|---|---|
Priority: | minor | Milestone: | unknown |
Component: | mythweb | Version: | unknown |
Severity: | low | Keywords: | |
Cc: | Ticket locked: | no |
Description
Because mod_perl reuses the same perl interpreter at times, the line
require "modules/$Path[0]/handler.pl";
only loads handler.pl once (the first time the code is reached), instead of every time the code is processes. As a result, calls after the first one yield nothing.
In addition, calling CORE::exit in handler.pl causes Apache to return an error. I recommend using a "goto" instead.
I'm attaching some recommended changes against Revision 15494.
Attachments (1)
Change History (4)
Changed 16 years ago by
Attachment: | mythweb.patch.txt added |
---|
comment:1 Changed 16 years ago by
FYI Sub, attaching patches with an extention of .diff or .patch causes them to be displayed really nicely and allows quick auditing much easier.
comment:2 Changed 16 years ago by
Actually, I can't do this. The whole reason for the CORE::exit call was because apache/mod_perl holds onto the stream process (in this case ffmpeg) even if the browser closes the session early, which eats up RAM and CPU resources.
However, after seeing your requested changes and realizing this, I've decided that the mod_perl code isn't ready for proper release, so I'll be removing it shortly.
Patch for handler.pl/mythweb.pl