Hi Mikael,
see my replies below.

Regards.

On 20/04/2017 10:03, Mikael Ekblom wrote:

Hi,

We have now implemented a pullaction, that is able to automatically generate usernames and passwords from a mssql database source, that is simulating our HR-system. So far so good.


Cool.

I can also see, that syncope is able to fetch start and end dates automatically from the db-source(date stated as datetime column), but, when we try to sync it the other way, we get an exception:

09:40:52.331 ERROR Error parsing value {0} of attribute Slutdatum:93 Method: handleAttribute

java.lang.NullPointerException: null

*First question:*

Is it now so, that syncope is able to transform and understand a datetime format quite easily into a correct java.date format when it pulls this information from an external source, but the conversion the other way round is troublesome when pushing/propagating data towards the external resource? Do I need to transform it first before the propagation? Maybe a SimpleDate formatter?


When Syncope fetches data from the db-source, we are doing pull.
Consider that such data is arriving in a format according to ConnId specifications; FYI, ConnId supports the following types:

https://github.com/Tirasa/ConnId/blob/connid-1.4.2.0/java/connector-framework/src/main/java/org/identityconnectors/framework/common/FrameworkUtil.java#L191-L212

where, as you can see, there is no Date.

This means that the date values arrive as text, and are parsed by Syncope according to the format specified for the mapped internal schema.

At the moment, however, during propagation the eventual conversion pattern is simply not taken into account:

https://github.com/apache/syncope/blob/2_0_X/core/provisioning-java/src/main/java/org/apache/syncope/core/provisioning/java/MappingManagerImpl.java#L257-L263

There should be room for an improvement here: would you mind to file an issue on JIRA about propagating Date values as formatted strings?

In that sense, it is better that it fails the other way around, because we are to fetch the values from the HR and they should not be managed from the syncope environment.


If such values are not to be used anywhere else in Syncope, then simply define the internal schema as String rather than Date.

*Second question:*

When a connector has the SEARCH and SYNC operations associated with itself: how is the logic supposed to work if you do have the change log column set and deletes a tuple in the external database? Pull mode is set to incremental. On our end here, even though we delete the tuple in the mssql hr simulation database, the associated account is not deleted within syncope. Not even with pull mode set to “Full reconciliation” funny enough.

What I can see, the pull action handler is not logging anything if the pull mode is not set to full reconciliation either. Syncope and the db do have the associated key values set correctly etc. The change log column is of a datetime format also. Does this has something to do with it?


You need to consider that everything related to SYNC mostly depends on how the actual connector implements such operation. The db-table connector is possibly one of the simplest out there; among its limitation, it is also not capable to feed Syncope with information about deleted users.

I would suggest to use the more flexible Scripted SQL for the job; you'll need, however, to provide your own Groovy scripts.

You might want to look at

https://github.com/apache/syncope/tree/2_0_X/fit/core-reference/src/test/resources/scriptedsql

as starting point; FYI, look how one of such sample scripts can generate DELETE events to send back to Syncope:

https://github.com/apache/syncope/blob/2_0_X/fit/core-reference/src/test/resources/scriptedsql/SyncScript.groovy#L78

by simply considering an additional 'deleted' column.

--
Francesco Chicchiriccò

Tirasa - Open Source Excellence
http://www.tirasa.net/

Member at The Apache Software Foundation
Syncope, Cocoon, Olingo, CXF, OpenJPA, PonyMail
http://home.apache.org/~ilgrosso/

Reply via email to