[jira] Updated: (ACE-61) DeploymentServlet - Split different purposes

["Toni Menzel (JIRA)" <ji...@apache.org>]

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

Toni Menzel updated ACE-61:
---------------------------

    Attachment: ACE-61.patch

Reorganized the class. 
Actually this should be done for basically all servlets to ease (still to do) writing unit tests.
At best, servlet logic should be really held to a bare minimum because HttpServletRequest/Response are ugly to mock.

> DeploymentServlet - Split different purposes
> --------------------------------------------
>
>                 Key: ACE-61
>                 URL: https://issues.apache.org/jira/browse/ACE-61
>             Project: Ace
>          Issue Type: Improvement
>            Reporter: Toni Menzel
>         Attachments: ACE-61.patch
>
>
> DeploymentServlet acts as an endpoint for two very different services:
> 1. serve available versions for a given gateway
> 2. ship a particular version
> It those different thing should not be handeled in one method as it makes things hard to understand.

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