[GitHub] [syncope] github-code-scanning[bot] commented on a diff in pull request #370: [SYNCOPE-1692] Refactoring the propagation process to allow for ConnId's updateDelta

2022-08-25 Thread GitBox
github-code-scanning[bot] commented on code in PR #370: URL: https://github.com/apache/syncope/pull/370#discussion_r954765216 ## core/persistence-api/src/main/java/org/apache/syncope/core/persistence/api/entity/task/PropagationData.java: ## @@ -0,0 +1,52 @@ +/* + * Licensed to

Re: [DISCUSS] Remove Camel Provisioning Manager

2022-08-25 Thread Andrea Patricelli
Hi all, Even if I agree with Misagh, I can confirm that even customers that were interested in the camel extension finally decided not to use it in favour of extending the default one. So agree on removing this feature on master. Just to know, we're going to upgrade camel deps and support it on

Re: [DISCUSS] Remove Camel Provisioning Manager

2022-08-25 Thread Fabio Martelli
Hi Francesco, I completely agree with you. I do think we should keep the entire project as simple as possible. So, removing unused and quite useless extensions could be the right approach. Regards, F. On Wed, Aug 24, 2022 at 1:16 PM Francesco Chicchiriccò wrote: > > Hi all, > working for

Re: [DISCUSS] Remove Camel Provisioning Manager

2022-08-25 Thread Francesco Chicchiriccò
On 24/08/22 15:58, Misagh Moayyed wrote: In general, I am in favor of this approach, but just to be on the safe side, would it make sense to deprecate the feature first? Just in case someone out there is using it, or would keeping the code around cause complications and maintenance issues?