ace-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bram de Kruijff (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (ACE-159) Register servlets under objectClass javax.servlet.Servlet instead of javax.servlet.http.HttpServlet
Date Thu, 28 Jul 2011 21:28:09 GMT

     [ https://issues.apache.org/jira/browse/ACE-159?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Bram de Kruijff updated ACE-159:
--------------------------------

    Attachment: ACE-159-toServletWithPropagate.patch

Ahhh yes. Turns out most code is already using this pattern. I was just looking at the exceptions
that failed. Attached a cumulative patch that resolves this as well.Basically this adds just
two changes:

1) Added propagate to the ace-webui-vaadin config dep
2) Mimic propagate behaviour in ace-log-servlet factory


> Register servlets under objectClass javax.servlet.Servlet instead of javax.servlet.http.HttpServlet
> ---------------------------------------------------------------------------------------------------
>
>                 Key: ACE-159
>                 URL: https://issues.apache.org/jira/browse/ACE-159
>             Project: Ace
>          Issue Type: Improvement
>    Affects Versions: 0.8.0
>            Reporter: Bram de Kruijff
>         Attachments: ACE-159-toServlet.patch, ACE-159-toServletWithPropagate.patch
>
>
> ACE registers its servlets whiteboard style under objectClass=javax.servlet.http.HttpServlet.
The custom whiteboard implementation HttpListener bridges them to available Http services.
> 1) I think strictly speaking this is not spec compliant as HttpServlet is not an interface?
> 2) It is unnecessarily specific as HttpService#registerServlet accepts javax.servlet.Servlet
> 3) It is not (out of the box) portable to Felix HTTP whiteboard or Amdatu Dispatcher
> For these reasons I'd suggest switching to objectClass=javax.servlet.Servlet. Would be
happy to work on a small patch if there is some consensus that this is a good idea.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Mime
View raw message