[jira] Issue Comment Edited: (PIVOT-402) Update QueryServlet API for consistency with client-side Query API

["Greg Brown (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/PIVOT-402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12828868#action_12828868 ] 

Greg Brown edited comment on PIVOT-402 at 2/3/10 12:54 AM:
-----------------------------------------------------------

API updates:
- Replaced custom server exceptions with existing QueryException.

- Added path argument to doGet(), doPost(), etc. that contains the part of the URL after the servlet path but before the query string (this is the part that is generally going to be most relevant to the servlet; the actual servlet path is still available via getServletPath()).

- Eliminated dependency on basic authentication (this is now up to the implementing class and can be facilitated by setting appropriate response headers and throwing QueryException(401)).

- Added a doPostAction() method - this allows a query servlet to perform additional "actions" in addition to the CRUD operations e.g. "log in" or "power on VM").

- Added prepare() and dispose() methods that are called before doGet(), doPost(), etc. that can be used to initialize a DB connection or otherwise.

- Moved determineContentLength to servlet initialization parameter (currently incomplete; property is hard-coded to false).

- Determine serializer to use based on Content-Type header rather than hard-coding (allows servlets to dynamically select serializer based on request; eventually this mapping will also be configurable via servlet init params).

Fixes:
- Added support for case-insensitive handling of keys in QueryDictionary (HTTP headers are case-insensitive, but query string parameters are not).


      was (Author: gbrown):
    API updates:
- Replaced custom server exceptions with existing QueryException.

- Added path argument to doGet(), doPost(), etc. that contains the part of the URL after the servlet path but before the query string (this is the part that is generally going to be most relevant to the servlet; the actual servlet path is still available via getServletPath()).

- Eliminated dependency on basic authentication (this is now up to the implementing class and can be facilitated by setting appropriate response headers and throwing QueryException(401)).

- Added a doPostAction() method - this allows a query servlet to perform additional "actions" in addition to the CRUD operations (it is a valid use of POST according to the HTTP spec, and allows callers to use POST do invoke operations such as "log in" or "power on VM").

- Added prepare() and dispose() methods that are called before doGet(), doPost(), etc. that can be used to initialize a DB connection or otherwise.

- Moved determineContentLength to servlet initialization parameter (currently incomplete; property is hard-coded to false).

- Determine serializer to use based on Content-Type header rather than hard-coding (allows servlets to dynamically select serializer based on request; eventually this mapping will also be configurable via servlet init params).

Fixes:
- Added support for case-insensitive handling of keys in QueryDictionary (HTTP headers are case-insensitive, but query string parameters are not).

  
> Update QueryServlet API for consistency with client-side Query API
> ------------------------------------------------------------------
>
>                 Key: PIVOT-402
>                 URL: https://issues.apache.org/jira/browse/PIVOT-402
>             Project: Pivot
>          Issue Type: Improvement
>          Components: web
>            Reporter: Greg Brown
>            Assignee: Greg Brown
>             Fix For: 1.4.1
>
>


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.