Issue1420

classification
Title: Make PyServlet use modules instead of file names
Type: behaviour Severity: normal
Components: None Versions: 2.5.1
Milestone:
process
Status: closed Resolution: wont fix
Dependencies: Superseder:
Assigned To: amak Nosy List: amak, istari
Priority: low Keywords: patch

Created on 2009-08-02.01:18:28 by istari, last changed 2012-03-29.18:37:21 by amak.

Files
File name Uploaded Description Edit Remove
trunk.patch istari, 2009-08-02.01:18:27
Messages
msg4979 (view) Author: Asgeir Bjarni Ingvarsson (istari) Date: 2009-08-02.01:18:27
Attached is a patch of a modification I made to PyServlet so that it
uses modules instead of file names.

With web.xml like this:
<servlet>
    <servlet-name>pyModuleServlet</servlet-name>
    <servlet-class>org.python.util.PyServlet</servlet-name>
</servlet>

<servlet-mapping>
    <servlet-name>pyModuleServlet</servlet-name>
    <url-pattern>/app/*</url-pattern>
</servlet-mapping>

A call to http://some.host/app/servlet will cause class servlet to be
served from war/WEB-INF/jython/app/servlet.py
msg6855 (view) Author: Alan Kennedy (amak) Date: 2012-03-19.20:13:50
We can't apply the patch as is, it breaks the existing behaviour that users may be relying on.

Also, the servlet is really only provided as an example: if the user requires different behaviour, they are free to customise the demo module, as you have. 

Modjy can use modules, have you considered using that?
msg6978 (view) Author: Alan Kennedy (amak) Date: 2012-03-29.18:37:21
Closing this as "won't fix".

As discussed in issue483305, PyServlet should really be considered a demo, i.e. an illustration of how to do servlet programming. As such, it should have been placed in the demo directory from the start. Also, changing its behaviour now also will break any existing usages.

We do have a standard WSGI-compatible servlet story in modjy, and we should be encouraging people to use that.
History
Date User Action Args
2012-03-29 18:37:21amaksetstatus: open -> closed
resolution: wont fix
messages: + msg6978
2012-03-19 20:13:50amaksetpriority: low
assignee: amak
messages: + msg6855
nosy: + amak
2009-08-02 01:18:29istaricreate