Bug 51423 - [Patch] to add a path and a version parameters to the list command
Summary: [Patch] to add a path and a version parameters to the list command
Status: RESOLVED WONTFIX
Alias: None
Product: Tomcat 7
Classification: Unclassified
Component: Manager (show other bugs)
Version: trunk
Hardware: All All
: P2 enhancement (vote)
Target Milestone: ---
Assignee: Tomcat Developers Mailing List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-06-23 07:53 UTC by Eiji Takahashi
Modified: 2018-05-17 09:30 UTC (History)
0 users



Attachments
patch for the Manager application (6.04 KB, patch)
2011-06-23 07:53 UTC, Eiji Takahashi
Details | Diff
patch (11.79 KB, patch)
2011-06-24 08:32 UTC, Eiji Takahashi
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Eiji Takahashi 2011-06-23 07:53:19 UTC
Created attachment 27196 [details]
patch for the Manager application

ManagerServlet and HTMLManagerServlet can only list all deployed applications.
("list" ant task too.)

So I made the patch that makes it possible to specify a context path and the version of this context as parameter.

If you specify the parameters as follows, the Manager application only displays one context information.

<<HTMLManagerServlet>>
http://localhost:8080/manager/html/list?path=/
http://localhost:8080/manager/html/list?path=/someapp&version=01

<<ManagerServlet>>
http://localhost:8080/manager/html?path=/someapp&version=01

And this feature was added to the "list" ant task too by this patch.

Regards.
Comment 1 Mark Thomas 2011-06-23 11:47:57 UTC
What is the use case for this? List is meant to list the deployed applications.
Comment 2 Eiji Takahashi 2011-06-24 08:30:37 UTC
Thanks for comment.

(In reply to comment #1)
> What is the use case for this? List is meant to list the deployed applications.
When you deploy many applications, it is a little bit hard searching the particular web application from the displayed list.
I think that some way to find particular web application is necessary.

---
And I'm trying to add the following adjustments to the patch.
- Undo the change to the list command
- Add show command for this feature.
- Add Textbox to HTMLManagerServlet for search for the applications.
Comment 3 Eiji Takahashi 2011-06-24 08:32:46 UTC
Created attachment 27203 [details]
patch
Comment 4 Mark Thomas 2011-06-28 16:43:58 UTC
I like the separate command but I'm not sure we need the search box on the HTML interface. If there are that many apps, I'd expect a user to just use the browser's built-in find functionality.
Comment 5 Mark Thomas 2018-05-17 09:30:54 UTC
The patch wasn't updated and, on reflection, using the browser's built-in 'find on page' can be used if required. I'm not convinced that the feature adds sufficient benefit to justify the additional code .