Bug 151078 - [patch] www/tomcat55: add http timeout for application
Summary: [patch] www/tomcat55: add http timeout for application
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Olli Hauer
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-09-29 17:50 UTC by Jason Helfman
Modified: 2010-10-07 22:19 UTC (History)
0 users

See Also:


Attachments
file.diff (508 bytes, patch)
2010-09-29 17:50 UTC, Jason Helfman
no flags Details | Diff
file.diff (1.16 KB, patch)
2010-09-29 17:50 UTC, Jason Helfman
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jason Helfman 2010-09-29 17:50:01 UTC
adding an http timeout to the application currently needs to be done by hand editing server.xml, and this would add a variable to configure this number via the port. the default is the vendor supplied number.
The server.xml patch should replace "tomcat55/files/patch-conf::server.xml" (created via make makepatch)
Comment 1 Olli Hauer freebsd_committer freebsd_triage 2010-09-30 06:24:26 UTC
Responsible Changed
From-To: freebsd-ports-bugs->ohauer

I take it. 
PR will be handled together with PR 150576 

@Jason, 
Do i see it right you will replace the port 8082 with 8009 (AJP) 
in this patch? 

There is already a patch for server.xml can you extend this patch instead?
Comment 2 Jason 2010-09-30 17:44:37 UTC
I didn't change the original Makefile for the AJP port it was pointing to by
default.

Creating a patch file of the server.xml patch file didn't work out as easily
as I thought it would, so I created a fresh one via 'make makepatch' and
substituted out the original '%%HTTP_PORT%%' and '%%AJP_1_3_PORT%%'. In
addition to this, I just added in '%%HTTP_TIMEOUT%%.

This patch file should work as a replacement of the current patch file for
server.xml.

My apologies if I wasn't more clear about that in the PR.

-jgh
Comment 3 Olli Hauer freebsd_committer freebsd_triage 2010-10-07 22:18:11 UTC
State Changed
From-To: open->closed

Patch was included with PR 150576