RE: [Discussion] New REST Service Interfaces - Entitlements

2013-01-18 Thread Jan Bernhardt
Hi Andrei, === Renaming Service === Rename AuthenticationController to EntitlementService(Impl), since containing methods have little to nothing to do with authentication. It is only about Entitlements... +1 for Entitlement, -1 for Service. I would keep names consistently, if use

RE: [Discussion] CXF migration (branches)

2013-01-18 Thread Jan Bernhardt
Hi Francesco, -Original Message- From: Francesco Chicchiriccò [mailto:ilgro...@apache.org] Sent: Donnerstag, 17. Januar 2013 17:47 To: dev@syncope.apache.org Subject: Re: [Discussion] CXF migration (branches) On 17/01/2013 10:54, Jan Bernhardt wrote: Hi Francesco, [...] If,

RE: [Discussion] New REST Service Interfaces - Entitlements

2013-01-18 Thread Jan Bernhardt
Btw. I would rather name the xml element Entitlement than EntitlementTO as the fact that it is a transfer object is not important on the xml level. I agree. Not using *TO ending would look nicer on transport layer. The question here is, should we also remove TO in Classnames, or just set a

Re: [Discussion] New REST Service Interfaces - Entitlements

2013-01-18 Thread Christian Schneider
I would rather do it in the annotations. Internally we often have a internal class without TO and a transfer object that ends in TO. So I think it makes sense to differentiate them on the class level. Christian On 18.01.2013 09:51, Jan Bernhardt wrote: Btw. I would rather name the xml element

RE: [Discussion] New REST Service Interfaces - Entitlements

2013-01-18 Thread Andrei Shakirin
Names will be Consistent. New CXF REST Services will all End with *Service. Old Spring Services currently ending with *Controller will eventually all vanish, once new Services are actually used (migration is complete). Than is absolutely OK for me. + 1. Andrei. Best regards. Jan

[jira] [Updated] (SYNCOPE-268) Enable Rest IntegrationTests to run more than once (per build)

2013-01-18 Thread Jan Bernhardt (JIRA)
[ https://issues.apache.org/jira/browse/SYNCOPE-268?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jan Bernhardt updated SYNCOPE-268: -- Assignee: Andrei Shakirin (was: Jan Bernhardt) Enable Rest IntegrationTests to run more

[jira] [Commented] (SYNCOPE-268) Enable Rest IntegrationTests to run more than once (per build)

2013-01-18 Thread Andrei Shakirin (JIRA)
[ https://issues.apache.org/jira/browse/SYNCOPE-268?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13557213#comment-13557213 ] Andrei Shakirin commented on SYNCOPE-268: - Actually all tests are re-runnable

[jira] [Commented] (SYNCOPE-252) Update LICENSE / NOTICE files for any added or removed Maven dependency

2013-01-18 Thread Jan Bernhardt (JIRA)
[ https://issues.apache.org/jira/browse/SYNCOPE-252?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13557214#comment-13557214 ] Jan Bernhardt commented on SYNCOPE-252: --- I decided to use Jackson instead of

[jira] [Commented] (SYNCOPE-252) Update LICENSE / NOTICE files for any added or removed Maven dependency

2013-01-18 Thread JIRA
[ https://issues.apache.org/jira/browse/SYNCOPE-252?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13557224#comment-13557224 ] Francesco Chicchiriccò commented on SYNCOPE-252: If you are only

[jira] [Commented] (SYNCOPE-231) Using Standard JAX-RS API in Syncope (Replace Spring Webservice Stack with Apache CXF)

2013-01-18 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/SYNCOPE-231?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13557241#comment-13557241 ] Hudson commented on SYNCOPE-231: Integrated in Syncope-trunk #452 (See

[jira] [Commented] (SYNCOPE-268) Enable Rest IntegrationTests to run more than once (per build)

2013-01-18 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/SYNCOPE-268?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13557240#comment-13557240 ] Hudson commented on SYNCOPE-268: Integrated in Syncope-trunk #452 (See

RE: Exception propagation in Rest interface: HTTP header vs body for details

2013-01-18 Thread Andrei Shakirin
I don't understand why WorkflowException should be mapped to INTERNAL_SERVER_ERROR: you get workflow exceptions when something is not allowed within the current workflow instance, for example. There is no error on the server, the error is on the client asking for a workflow action not

[jira] [Commented] (SYNCOPE-252) Update LICENSE / NOTICE files for any added or removed Maven dependency

2013-01-18 Thread Jan Bernhardt (JIRA)
[ https://issues.apache.org/jira/browse/SYNCOPE-252?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13557253#comment-13557253 ] Jan Bernhardt commented on SYNCOPE-252: --- Yes, you are right,

[jira] [Commented] (SYNCOPE-252) Update LICENSE / NOTICE files for any added or removed Maven dependency

2013-01-18 Thread JIRA
[ https://issues.apache.org/jira/browse/SYNCOPE-252?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13557261#comment-13557261 ] Francesco Chicchiriccò commented on SYNCOPE-252: About javax.ws.rs-api,

[jira] [Commented] (SYNCOPE-231) Using Standard JAX-RS API in Syncope (Replace Spring Webservice Stack with Apache CXF)

2013-01-18 Thread Hudson (JIRA)
[ https://issues.apache.org/jira/browse/SYNCOPE-231?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13557307#comment-13557307 ] Hudson commented on SYNCOPE-231: Integrated in Syncope-trunk #454 (See

[jira] [Commented] (SYNCOPE-280) Update some checked exceptions to runtime ones

2013-01-18 Thread Andrei Shakirin (JIRA)
[ https://issues.apache.org/jira/browse/SYNCOPE-280?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13557319#comment-13557319 ] Andrei Shakirin commented on SYNCOPE-280: - InvalidPolicyType will be also

[jira] [Resolved] (SYNCOPE-268) Enable Rest IntegrationTests to run more than once (per build)

2013-01-18 Thread Andrei Shakirin (JIRA)
[ https://issues.apache.org/jira/browse/SYNCOPE-268?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Andrei Shakirin resolved SYNCOPE-268. - Resolution: Fixed Enable Rest IntegrationTests to run more than once (per build)

[jira] [Assigned] (SYNCOPE-244) Make external property file usage possible

2013-01-18 Thread JIRA
[ https://issues.apache.org/jira/browse/SYNCOPE-244?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Francesco Chicchiriccò reassigned SYNCOPE-244: -- Assignee: Francesco Chicchiriccò Make external property file