[jboss-jira] [JBoss JIRA] Closed: (JBAS-4858) SecurityDeployer changes with metadata

[Anil Saldhana (JIRA)]

     [ http://jira.jboss.com/jira/browse/JBAS-4858?page=all ]

Anil Saldhana closed JBAS-4858.
-------------------------------

    Resolution: Done

Revisions 7009 and 7010.

> SecurityDeployer changes with metadata
> --------------------------------------
>
>                 Key: JBAS-4858
>                 URL: http://jira.jboss.com/jira/browse/JBAS-4858
>             Project: JBoss Application Server
>          Issue Type: Task
>      Security Level: Public(Everyone can see) 
>          Components: Security
>    Affects Versions: JBossAS-5.0.0.Beta2
>            Reporter: Anil Saldhana
>         Assigned To: Anil Saldhana
>             Fix For: JBossAS-5.0.0.CR1
>
>
> The Security Deployer needs to be changed as per the following rant from Adrian
> =============================
> To me this is the wrong approach anyway.
> There shouldn't be any code that does
> if (ejbdeployment)
>   doThis();
> else if (webdeployment)
>   doThat();
> else if (unknowntype) // OOPS (pun intended  ;-) 
>   cantDoIt();
> The webservice deployers should be working off some generic metadata that triggers them to operate.
> Then each type of deployment can populate that metadata to say "I want a webservice endpoint for this".
> This is a seperate deployer for each type we know how to map to a webservice it takes the ejb/web/other metadata and maps or bridges it to the webservice metadata.
> It shouldn't just be restricted to ejbs and wars.
> e.g. An MBean should be capable of being an endpoint if there is a deployer that creates the relevant metadata attachment from say a META-INF/jboss-webservice.xml in the sar.
> The security deployers take the same brain dead (non object orientated) approach.
> ============================================================

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira